🐛 fix(execute): keep spaces in quoted shebang interpreter paths (#4064)

Author: r3wretrhy

docs/changelog/4064.bugfix.rst

@@ -0,0 +1,2 @@
+Shebang parsing now uses ``shlex``, so a quoted interpreter path that contains spaces is kept as one argument when
+``TOX_LIMITED_SHEBANG`` rewrites the invocation - by :user:`r3wretrhy`.

src/tox/execute/util.py

@@ -1,5 +1,6 @@
 from __future__ import annotations
 
+import shlex
 from pathlib import Path
 
 
@@ -25,7 +26,11 @@ def shebang(exe: str) -> list[str] | None:
         decoded = shebang_line.decode("UTF-8")
     except UnicodeDecodeError:
         return None
-    return [i.strip() for i in decoded.strip().split() if i.strip()]
+    try:
+        parts = shlex.split(decoded.strip(), posix=True)
+    except ValueError:
+        return None
+    return parts or None
 
 
 __all__ = [

tests/execute/local_subprocess/test_execute_util.py

@@ -14,6 +14,12 @@ def test_shebang_found(tmp_path: Path) -> None:
     assert shebang(str(script_path)) == ["/bin/python", "-c"]
 
 
+def test_shebang_quoted_interpreter_path(tmp_path: Path) -> None:
+    script_path = tmp_path / "a"
+    script_path.write_text('#! "/Program Files/Python/python.exe" -c\n', encoding="utf-8")
+    assert shebang(str(script_path)) == ["/Program Files/Python/python.exe", "-c"]
+
+
 def test_shebang_file_missing(tmp_path: Path) -> None:
     script_path = tmp_path / "a"
     assert shebang(str(script_path)) is None