[IMP] tools/compile23.py

Recent systems don't have python2 installed.
Let the pre-commit hook pass, but warns, when it must verify scripts that
should be compatible with python2.

closes #280

Signed-off-by: Christophe Simonis (chs) <[email protected]>
Co-authored-by: Christophe Simonis (chs) <[email protected]>

Author: cawo-odoo

.pre-commit-config.yaml

@@ -5,6 +5,8 @@ repos:
         name: Compile python files using the expected runtime version
         entry: ./tools/compile23.py
         language: script
+        require_serial: true
+        verbose: true
       - id: bad-import-000
         name: Incompatible import with old versions in tests and `0.0.0` scripts
         language: pygrep

tools/compile23.py

@@ -3,6 +3,7 @@
 import subprocess
 import sys
 from pathlib import PurePath
+from shutil import which
 
 py2_only_patterns = []
 py2_files = []
@@ -41,9 +42,20 @@
 
 
 if py2_files:
-    s = subprocess.run(["python2", "-m", "compileall", "-f", "-q", *py2_files], check=False)
-    if s.returncode:
-        rc = 1
+    if which("python2"):
+        s = subprocess.run(["python2", "-m", "compileall", "-f", "-q", *py2_files], check=False)
+        if s.returncode:
+            rc = 1
+    else:
+        lines = [
+            "WARNING: `python2` hasn't been found in $PATH",
+            "You must ensure the following files are compatible with python2:",
+            *[f" - {f}" for f in py2_files],
+        ]
+        width = max(map(len, lines))
+        message = "\n".join(f"@ {line: <{width}s} @" for line in lines)
+        extra = "@" * (width + 4)
+        print(f"{extra}\n{message}\n{extra}", file=sys.stderr)
 
 if py3_files:
     s = subprocess.run(["python3", "-m", "compileall", "-f", "-q", *py3_files], check=False)