[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