Skip to contentSkip to conversation

scripts/clang-tools: switch explicitly to Python 3

PATCH In review Masahiro Yamada 3 participants 3 messages Feb 2, 2021 – Feb 3, 2021

Subsystems (2)
Kbuild Clang/LLVM build support
lore
Share & tools
lore

Masahiro Yamada

author
#0 lore raw

PATCH v2 1/1 scripts/clang-tools: switch explicitly to Python 3

For the same reason as commit 51839e29cb59 ("scripts: switch explicitly to Python 3"), switch some more scripts, which I tested and confirmed working on Python 3.

Signed-off-by Masahiro Yamada <[email protected]>
Acked-by Nathan Chancellor <[email protected]>

---

Changes in v2:
- Drop the spdxcheck.py change. The same fix exists in linux-next.

 scripts/clang-tools/gen_compile_commands.py | 2 +-
 scripts/clang-tools/run-clang-tools.py      | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)
scripts/clang-tools/gen_compile_commands.py +1 -1
index 19963708bcf8..8ddb5d099029 100755
@@ -1,4 +1,4 @@
-#!/usr/bin/env python
+#!/usr/bin/env python3
 # SPDX-License-Identifier: GPL-2.0
 #
 # Copyright (C) Google LLC, 2018
scripts/clang-tools/run-clang-tools.py +1 -1
index fa7655c7cec0..f754415af398 100755
@@ -1,4 +1,4 @@
-#!/usr/bin/env python
+#!/usr/bin/env python3
 # SPDX-License-Identifier: GPL-2.0
 #
 # Copyright (C) Google LLC, 2020
Signature · 1 line
2.27.0
#1 lore raw
Quoted 5 lines · Masahiro Yamada

For the same reason as commit 51839e29cb59 ("scripts: switch explicitly to Python 3"), switch some more scripts, which I tested and confirmed working on Python 3.

Quoted 7 lines
Signed-off-by Masahiro Yamada <[email protected]>
Acked-by Nathan Chancellor <[email protected]>

---

Changes in v2:
- Drop the spdxcheck.py change. The same fix exists in linux-next.

Link https://lore.kernel.org/r/[email protected] open here

- Sedat -

Quoted 31 lines
 scripts/clang-tools/gen_compile_commands.py | 2 +-
 scripts/clang-tools/run-clang-tools.py      | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)
scripts/clang-tools/gen_compile_commands.py +1 -1
index 19963708bcf8..8ddb5d099029 100755
@@ -1,4 +1,4 @@
-#!/usr/bin/env python
+#!/usr/bin/env python3
 # SPDX-License-Identifier: GPL-2.0
 #
 # Copyright (C) Google LLC, 2018
scripts/clang-tools/run-clang-tools.py +1 -2
index fa7655c7cec0..f754415af398 100755
@@ -1,4 +1,4 @@
-#!/usr/bin/env python
+#!/usr/bin/env python3
 # SPDX-License-Identifier: GPL-2.0
 #
 # Copyright (C) Google LLC, 2020
--

2.27.0

--
You received this message because you are subscribed to the Google Groups "Clang Built Linux" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To view this discussion on the web visit https://groups.google.com/d/msgid/clang-built-linux/20210202060604.711160-1-masahiroy%40kernel.org.

reviewer Reviewed-by on v2 1/1
#2 lore raw
In reply to Masahiro Yamada
Quoted 7 lines · Masahiro Yamada

For the same reason as commit 51839e29cb59 ("scripts: switch explicitly to Python 3"), switch some more scripts, which I tested and confirmed working on Python 3.

Signed-off-by Masahiro Yamada <[email protected]>
Acked-by Nathan Chancellor <[email protected]>

Thanks Masahiro.

Reviewed-by Nick Desaulniers <[email protected]>
Quoted 32 lines

---

Changes in v2:
- Drop the spdxcheck.py change. The same fix exists in linux-next.

 scripts/clang-tools/gen_compile_commands.py | 2 +-
 scripts/clang-tools/run-clang-tools.py      | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)
scripts/clang-tools/gen_compile_commands.py +1 -1
index 19963708bcf8..8ddb5d099029 100755
@@ -1,4 +1,4 @@
-#!/usr/bin/env python
+#!/usr/bin/env python3
 # SPDX-License-Identifier: GPL-2.0
 #
 # Copyright (C) Google LLC, 2018
scripts/clang-tools/run-clang-tools.py +1 -2
index fa7655c7cec0..f754415af398 100755
@@ -1,4 +1,4 @@
-#!/usr/bin/env python
+#!/usr/bin/env python3
 # SPDX-License-Identifier: GPL-2.0
 #
 # Copyright (C) Google LLC, 2020
--

2.27.0

Signature · 2 lines
Thanks,
~Nick Desaulniers