Skip to content

[3.8] bpo-34886: Fix subprocess.run handling of exclusive arguments (GH-11727)#13916

Merged
miss-islington merged 1 commit into
python:3.8from
miss-islington:backport-8cc605a-3.8
Jun 8, 2019
Merged

[3.8] bpo-34886: Fix subprocess.run handling of exclusive arguments (GH-11727)#13916
miss-islington merged 1 commit into
python:3.8from
miss-islington:backport-8cc605a-3.8

Conversation

@miss-islington
Copy link
Copy Markdown
Contributor

@miss-islington miss-islington commented Jun 8, 2019

Fix an unintended ValueError from :func:subprocess.run when checking for
conflicting input and stdin or capture_output and stdout or stderr args
when they were explicitly provided but with None values within a passed in
**kwargs dict rather than as passed directly by name.
(cherry picked from commit 8cc605a)

Co-authored-by: Rémi Lapeyre remi.lapeyre@henki.fr

https://bugs.python.org/issue34886

…GH-11727)

Fix an unintended ValueError from :func:`subprocess.run` when checking for
conflicting `input` and `stdin` or `capture_output` and `stdout` or `stderr` args
when they were explicitly provided but with `None` values within a passed in
`**kwargs` dict rather than as passed directly by name.
(cherry picked from commit 8cc605a)

Co-authored-by: Rémi Lapeyre <remi.lapeyre@henki.fr>
@miss-islington
Copy link
Copy Markdown
Contributor Author

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

@miss-islington miss-islington merged commit 6324ac1 into python:3.8 Jun 8, 2019
@miss-islington miss-islington deleted the backport-8cc605a-3.8 branch June 8, 2019 15:15
@miss-islington
Copy link
Copy Markdown
Contributor Author

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

1 similar comment
@miss-islington
Copy link
Copy Markdown
Contributor Author

@remilapeyre and @gpshead: 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-bug An unexpected behavior, bug, or error

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants