Skip to content

[3.7] bpo-34812: subprocess._args_from_interpreter_flags(): add isolated (GH-10675)#10684

Merged
miss-islington merged 1 commit into
python:3.7from
miss-islington:backport-9de3632-3.7
Nov 23, 2018
Merged

[3.7] bpo-34812: subprocess._args_from_interpreter_flags(): add isolated (GH-10675)#10684
miss-islington merged 1 commit into
python:3.7from
miss-islington:backport-9de3632-3.7

Conversation

@miss-islington

@miss-islington miss-islington commented Nov 23, 2018

Copy link
Copy Markdown
Contributor

The "-I" command line option (run Python in isolated mode) is now
also copied by the multiprocessing and distutils modules when
spawning child processes. Previously, only -E and -s options (enabled
by -I) were copied.

subprocess._args_from_interpreter_flags() now copies the -I flag.
(cherry picked from commit 9de3632)

Co-authored-by: Victor Stinner vstinner@redhat.com

https://bugs.python.org/issue34812

…ythonGH-10675)

The "-I" command line option (run Python in isolated mode) is now
also copied by the multiprocessing and distutils modules when
spawning child processes. Previously, only -E and -s options (enabled
by -I) were copied.

subprocess._args_from_interpreter_flags() now copies the -I flag.
(cherry picked from commit 9de3632)

Co-authored-by: Victor Stinner <vstinner@redhat.com>

@vstinner vstinner left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, good bot.

@miss-islington

Copy link
Copy Markdown
Contributor Author

@vstinner: Status check is done, and it's a success ✅ .

@miss-islington miss-islington merged commit 01e5799 into python:3.7 Nov 23, 2018
@miss-islington miss-islington deleted the backport-9de3632-3.7 branch November 23, 2018 17:13
@miss-islington

Copy link
Copy Markdown
Contributor Author

@vstinner: Status check is done, and it's a success ✅ .

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

Labels

type-security A security issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants