[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PULL 31/46] scripts/checkpatch.pl: Only allow Python 3 interpreter
From: |
Philippe Mathieu-Daudé |
Subject: |
[PULL 31/46] scripts/checkpatch.pl: Only allow Python 3 interpreter |
Date: |
Thu, 6 Feb 2020 22:19:21 +0100 |
Since commit ddf9069963 QEMU requires Python >= 3.5.
PEP 0394 [*] states that 'python3' should be available and
that 'python' is optional.
To avoid problem with unsupported versions, enforce the
shebang interpreter to Python 3.
[*] https://www.python.org/dev/peps/pep-0394/
Reported-by: John Snow <address@hidden>
Acked-by: Paolo Bonzini <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
---
scripts/checkpatch.pl | 6 ++++++
1 file changed, 6 insertions(+)
diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
index 3aef6e3dfe..ce43a306f8 100755
--- a/scripts/checkpatch.pl
+++ b/scripts/checkpatch.pl
@@ -1460,6 +1460,12 @@ sub process {
}
}
+# Only allow Python 3 interpreter
+ if ($realline == 1 &&
+ $line =~ /^\+#!\ *\/usr\/bin\/(?:env )?python$/) {
+ ERROR("please use python3 interpreter\n" . $herecurr);
+ }
+
# Accept git diff extended headers as valid patches
if ($line =~ /^(?:rename|copy) (?:from|to) [\w\/\.\-]+\s*$/) {
$is_patch = 1;
--
2.21.1
- [PULL 23/46] tests/acceptance/boot_linux_console: Do not use VGA on Clipper machine, (continued)
- [PULL 23/46] tests/acceptance/boot_linux_console: Do not use VGA on Clipper machine, Philippe Mathieu-Daudé, 2020/02/06
- [PULL 21/46] tests/acceptance/virtio_check_params: Default to -nodefaults, Philippe Mathieu-Daudé, 2020/02/06
- [PULL 19/46] tests/acceptance/virtio_check_params: Improve exception logging, Philippe Mathieu-Daudé, 2020/02/06
- [PULL 22/46] tests/acceptance/virtio_check_params: Disable the test, Philippe Mathieu-Daudé, 2020/02/06
- [PULL 25/46] tests/acceptance/migration: Factor out assert_migration(), Philippe Mathieu-Daudé, 2020/02/06
- [PULL 27/46] tests/acceptance/migration: Test UNIX transport when migrating, Philippe Mathieu-Daudé, 2020/02/06
- [PULL 26/46] tests/acceptance/migration: Factor out do_migrate(), Philippe Mathieu-Daudé, 2020/02/06
- [PULL 28/46] tests/acceptance/migration: Test EXEC transport when migrating, Philippe Mathieu-Daudé, 2020/02/06
- [PULL 29/46] tests/acceptance/migration: Add the 'migration' tag, Philippe Mathieu-Daudé, 2020/02/06
- [PULL 30/46] tests/acceptance/migration: Default to -nodefaults, Philippe Mathieu-Daudé, 2020/02/06
- [PULL 31/46] scripts/checkpatch.pl: Only allow Python 3 interpreter,
Philippe Mathieu-Daudé <=
- [PULL 32/46] tests/qemu-iotests/check: Allow use of python3 interpreter, Philippe Mathieu-Daudé, 2020/02/06
- [PULL 33/46] tests/qemu-iotests: Explicit usage of Python 3 (scripts with __main__), Philippe Mathieu-Daudé, 2020/02/06
- [PULL 34/46] tests: Explicit usage of Python 3, Philippe Mathieu-Daudé, 2020/02/06
- [PULL 36/46] scripts/minikconf: Explicit usage of Python 3, Philippe Mathieu-Daudé, 2020/02/06
- [PULL 35/46] scripts: Explicit usage of Python 3 (scripts with __main__), Philippe Mathieu-Daudé, 2020/02/06
- [PULL 37/46] scripts/tracetool: Remove shebang header, Philippe Mathieu-Daudé, 2020/02/06
- [PULL 38/46] tests/acceptance: Remove shebang header, Philippe Mathieu-Daudé, 2020/02/06
- [PULL 39/46] tests/vm: Remove shebang header, Philippe Mathieu-Daudé, 2020/02/06
- [PULL 40/46] tests/qemu-iotests: Explicit usage of Python3 (scripts without __main__), Philippe Mathieu-Daudé, 2020/02/06
- [PULL 41/46] scripts: Explicit usage of Python 3 (scripts without __main__), Philippe Mathieu-Daudé, 2020/02/06