[PATCH] D76768: [analyzer] Added support of scan-build and exploded-graph-rewriter regression tests for Windows
Nico Weber via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Apr 10 03:12:26 PDT 2020
thakis added a comment.
In D76768#1971420 <https://reviews.llvm.org/D76768#1971420>, @ASDenysPetrov wrote:
> @thakis
>
> > 'perl' is not recognized as an internal or external command,
>
> I think you got another problem, that Perl isn't just presented in your PATH. It is not the case for buildbot logs.
I tried again in a `git bash` shell where perl is present. With that, I get this error: http://codepad.org/t6ZhaCXI
"Can't find scan-build on PATH."
I guess that means that the bat file runs, but the perl -s scan-build therein fails? scan-build is right next to the bat file, but the perl process can't know the directory the .bat file is in. With this change it gets a bit farther:
-perl -S scan-build %*
+perl -S %~dp0\scan-build %*
(Since it works for others I probably misunderstood how things are supposed to work, and this change might be wrong.)
New error:
"""
command stderr:
===============
Can't find c:\src\llvm-project\clang\tools\scan-build\bin\\scan-build on PATH, '.' not in PATH.
"""
If I also remove the `-S` switch, that goes away (what's supposed to add clang\tools\scan-build\bin to path? Or is it supposed to be copied to bin\ in the out dir? It looks like clang\tools\scan-build\CMakeList.txt has logic to copy the script around, but clang\test\CMakeLists.txt doesn't have a dependency on the scan-build target, so building `check-clang` doesn't guarantee that the script is copied. Maybe it's just this dependency that's missing?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D76768/new/
https://reviews.llvm.org/D76768
More information about the cfe-commits
mailing list