Open llvmbot opened 8 years ago
If we can get testing for this patch by several Windows users, I'd suggest committing the patch in tree.
Non-reversed diff:
diff --git a/tools/scan-build/CMakeLists.txt b/tools/scan-build/CMakeLists.txt index 78c243d8e0..a3983d584c 100644 --- a/tools/scan-build/CMakeLists.txt +++ b/tools/scan-build/CMakeLists.txt @@ -2,15 +2,16 @@ option(CLANG_INSTALL_SCANBUILD "Install the scan-build tool" ON)
include(GNUInstallDirs)
+set(BinFiles
if (WIN32 AND NOT CYGWIN)
I noticed this too. I wrote the following patch, but I haven't tested it extensively:
diff --git a/tools/scan-build/CMakeLists.txt b/tools/scan-build/CMakeLists.txt index a3983d584c..78c243d8e0 100644 --- a/tools/scan-build/CMakeLists.txt +++ b/tools/scan-build/CMakeLists.txt @@ -2,16 +2,15 @@ option(CLANG_INSTALL_SCANBUILD "Install the scan-build tool" ON)
include(GNUInstallDirs)
-set(BinFiles
if (WIN32 AND NOT CYGWIN)
assigned to @tkremenek
Extended Description
I follow the latest set of instructions to download, configure and build llvm/clang 3.9.0 (trunk 267376) on an up-to-date windows10 system with Visual Studio 2013.
I find that the perl scripts for scan-build and ccc-analyzer are not installed. The .bat files, which act as a trampoline through perl into the actual perl code is installed.
I have been able to manually install a plausible scan-build perl script in place, compensate for installation path names, but then find I have to do the same thing for ccc-analyzer, at which point it starts getting tedious.