[PATCH] D101139: Create install targets for scan-build-py.

Anders Waldenborg via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon May 3 07:12:27 PDT 2021


wanders added inline comments.


================
Comment at: clang/tools/scan-build-py/CMakeLists.txt:8
+     "bin/intercept-cc"
+     "bin/scan-build")
+
----------------
This overwrites the "bin/scan-build" that was installed from the scan-build (no -py) subdirectory.

So with this  "scan-build" will suddenly mean the python variant instead of the per variant. That might be fine (?), but can't be a good idea to have that done by first installing the perl scan-build and then overwriting it with the python scan-build.  Also the man-page that the perl variant installs is not overwritten so "man scan-build" will show manual page for the perl variant.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D101139/new/

https://reviews.llvm.org/D101139



More information about the cfe-commits mailing list