[all-commits] [llvm/llvm-project] 4c1e48: scan-build-py: Change scripts to explicitly requir...
Anders Waldenborg via All-commits
all-commits at lists.llvm.org
Thu Jun 2 11:10:18 PDT 2022
Branch: refs/heads/main
Home: https://github.com/llvm/llvm-project
Commit: 4c1e487c413810655757ea19d0cf3d4f5c40898e
https://github.com/llvm/llvm-project/commit/4c1e487c413810655757ea19d0cf3d4f5c40898e
Author: Anders Waldenborg <anders at 0x63.nu>
Date: 2022-06-02 (Thu, 02 Jun 2022)
Changed paths:
M clang/tools/scan-build-py/bin/analyze-build
M clang/tools/scan-build-py/bin/intercept-build
M clang/tools/scan-build-py/bin/scan-build
M clang/tools/scan-build-py/libexec/analyze-c++
M clang/tools/scan-build-py/libexec/analyze-cc
M clang/tools/scan-build-py/libexec/intercept-c++
M clang/tools/scan-build-py/libexec/intercept-cc
Log Message:
-----------
scan-build-py: Change scripts to explicitly require python3
The "#!" line in all scan-build-py scripts were using just bare
"/usr/bin/python" which according to PEP-0394 can be either python3,
python2 or not exist at all.
E.g in latest debian and ubuntu releases "/usr/bin/python" does not
exist at all by default and user must install python-is-python2 or
python-is-python3 packages to get the bare version less "python"
command.
Until recently (70b06fe8a186 "scan-build-py: Force the opening in utf-8"
changed "libscanbuild") these scripts worked in both python2 and
python3, but now they (rightfully) are python3 only, and broke on
systems where the "python" command means python2.
By changing the "#!" to be "python3" it is not only explicit that the
scripts require python3 it also works on systems where "python" command
is python2 or nonexistent.
Differential Revision: https://reviews.llvm.org/D126804
More information about the All-commits
mailing list