Skip to content

[3.15] gh-152548: Add options, env and timeout parameters to runInSubprocess() (GH-155163) - #155238

Open
serhiy-storchaka wants to merge 2 commits into
python:3.15from
serhiy-storchaka:backport-1d90627-3.15
Open

[3.15] gh-152548: Add options, env and timeout parameters to runInSubprocess() (GH-155163)#155238
serhiy-storchaka wants to merge 2 commits into
python:3.15from
serhiy-storchaka:backport-1d90627-3.15

Conversation

@serhiy-storchaka

Copy link
Copy Markdown
Member

Manual backport of GH-155163; miss-islington could not apply it cleanly because
GH-155167 (the backport of GH-155162) has not landed on 3.15 yet. This branch is
based on GH-155167 and should be merged after it.

Add three keyword-only parameters to runInSubprocess():

  • options -- interpreter command line options for the subprocess.
  • env -- environment variables, on top of the inherited environment; a value
    of None unsets a variable.
  • timeout -- how long to wait for the subprocess; the test is reported as an
    error if it does not complete in time. There is no timeout by default.

(cherry picked from commit 1d90627)

serhiy-storchaka and others added 2 commits August 4, 2026 10:03
… the test (pythonGH-155162)

The subprocess writes the result of the test before exiting, so a crash
during interpreter finalization left the result intact and the test was
reported as passed.  Check the exit code too, after replaying the outcomes,
so that a failure of the test itself is still reported as a failure.
(cherry picked from commit ef0affb)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
…rocess() (pythonGH-155163)

They run the test subprocess with specific interpreter command line options
and environment variables, and limit how long it may take.  All are
keyword-only.  env is layered over the inherited environment; a None value
unsets a variable.  There is no timeout by default.
(cherry picked from commit 1d90627)

Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
@read-the-docs-community

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant