[Python-checkins] gh-90473: Fail subprocess early on Emscripten/WASI (GH-92802)
tiran
webhook-mailer at python.org
Sat May 14 14:23:24 EDT 2022
https://github.com/python/cpython/commit/db0b455ff482df68f331411bf22b3e5829398280
commit: db0b455ff482df68f331411bf22b3e5829398280
branch: main
author: Christian Heimes <christian at python.org>
committer: tiran <christian at python.org>
date: 2022-05-14T20:23:19+02:00
summary:
gh-90473: Fail subprocess early on Emscripten/WASI (GH-92802)
files:
A Misc/NEWS.d/next/Library/2022-05-14-11-41-23.gh-issue-90473.kPdOZl.rst
M Lib/subprocess.py
diff --git a/Lib/subprocess.py b/Lib/subprocess.py
index 6e61cc2e5e7b0..e10b01047ebef 100644
--- a/Lib/subprocess.py
+++ b/Lib/subprocess.py
@@ -74,6 +74,9 @@
else:
_mswindows = True
+# wasm32-emscripten and wasm32-wasi do not support processes
+_can_fork_exec = sys.platform not in {"emscripten", "wasi"}
+
if _mswindows:
import _winapi
from _winapi import (CREATE_NEW_CONSOLE, CREATE_NEW_PROCESS_GROUP,
@@ -97,13 +100,10 @@
"CREATE_NO_WINDOW", "DETACHED_PROCESS",
"CREATE_DEFAULT_ERROR_MODE", "CREATE_BREAKAWAY_FROM_JOB"])
else:
- if sys.platform in {"emscripten", "wasi"}:
- def _fork_exec(*args, **kwargs):
- raise OSError(
- errno.ENOTSUP, f"{sys.platform} does not support processes."
- )
- else:
+ if _can_fork_exec:
from _posixsubprocess import fork_exec as _fork_exec
+ else:
+ _fork_exec = None
import select
import selectors
@@ -801,6 +801,11 @@ def __init__(self, args, bufsize=-1, executable=None,
encoding=None, errors=None, text=None, umask=-1, pipesize=-1,
process_group=None):
"""Create new Popen instance."""
+ if not _can_fork_exec:
+ raise OSError(
+ errno.ENOTSUP, f"{sys.platform} does not support processes."
+ )
+
_cleanup()
# Held while anything is calling waitpid before returncode has been
# updated to prevent clobbering returncode if wait() or poll() are
diff --git a/Misc/NEWS.d/next/Library/2022-05-14-11-41-23.gh-issue-90473.kPdOZl.rst b/Misc/NEWS.d/next/Library/2022-05-14-11-41-23.gh-issue-90473.kPdOZl.rst
new file mode 100644
index 0000000000000..bf5ee542182e0
--- /dev/null
+++ b/Misc/NEWS.d/next/Library/2022-05-14-11-41-23.gh-issue-90473.kPdOZl.rst
@@ -0,0 +1,2 @@
+:mod:`subprocess` now fails early on Emscripten and WASI platforms to work
+around missing :func:`os.pipe` on WASI.
More information about the Python-checkins
mailing list