FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

[3.8] bpo-43423: Fix IndexError in subprocess _communicate function (GH-24777) by miss-islington · Pull Request #24824 · python/cpython · GitHub

Repository navigation

[3.8] bpo-43423: Fix IndexError in subprocess _communicate function (GH-24777) - #24824

Closed
miss-islington wants to merge 1 commit into
python:3.8from
miss-islington:backport-b4fc44b-3.8
Closed

miss-islington wants to merge 1 commit into
python:3.8from
miss-islington:backport-b4fc44b-3.8

Conversation

miss-islington commented Mar 11, 2021 •
edited
Loading

Copy link
Copy Markdown
Contributor

Check to make sure stdout and stderr are not empty before selecting an item from them in Windows subprocess._communicate.

Co-authored-by: Gregory P. Smith greg@krypto.org
(cherry picked from commit b4fc44b)

Co-authored-by: Chris Griffith chris@cdgriffith.com

https://bugs.python.org/issue43423

Automerge-Triggered-By: GH:gpshead

…H-24777)

Check to make sure stdout and stderr are not empty before selecting an item from them in Windows subprocess._communicate.

Co-authored-by: Gregory P. Smith <greg@krypto.org>
(cherry picked from commit b4fc44b)

Co-authored-by: Chris Griffith <chris@cdgriffith.com>

Copy link
Copy Markdown
Contributor Author

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

gpshead changed the title [3.8] bpo-43423 Fix IndexError in subprocess _communicate function (GH-24777) [3.8] bpo-43423: Fix IndexError in subprocess _communicate function (GH-24777) Mar 12, 2021

Copy link
Copy Markdown
Contributor Author

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

1 similar comment

Copy link
Copy Markdown
Contributor Author

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

gpshead closed this Mar 12, 2021
miss-islington deleted the backport-b4fc44b-3.8 branch March 12, 2021 01:31
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting merge type-bug An unexpected behavior, bug, or error

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants


Back | FazBrowse Home | New Git URL