[3.7] bpo-36719: Fix regrtest MultiprocessThread (GH-13301) by miss-islington · Pull Request #13303 · python/cpython
-
-
Notifications
You must be signed in to change notification settings - Fork 34.8k
Merged
Conversation
Contributor
MultiprocessThread.kill() now closes stdout and stderr to prevent
popen.communicate() to hang.
(cherry picked from commit c923c34)
Co-authored-by: Victor Stinner vstinner@redhat.com
MultiprocessThread.kill() now closes stdout and stderr to prevent popen.communicate() to hang. (cherry picked from commit c923c34) Co-authored-by: Victor Stinner <vstinner@redhat.com>
the-knights-who-say-ni
added
the
CLA signed
label
bedevere-bot
mentioned this pull request
bedevere-bot
added
skip news
awaiting review
labels
vstinner
approved these changes
vstinner
left a comment
vstinner
left a comment
Member
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, good bot.
bedevere-bot
added
awaiting merge
and removed
awaiting review
labels
miss-islington
commented
May 14, 2019
miss-islington commented
Contributor Author
@vstinner: Status check is done, and it's a failure ❌ .
2 similar comments
miss-islington
commented
May 14, 2019
miss-islington commented
Contributor Author
@vstinner: Status check is done, and it's a failure ❌ .
miss-islington
commented
May 14, 2019
miss-islington commented
Contributor Author
@vstinner: Status check is done, and it's a failure ❌ .
vstinner
merged commit
d8e123a
into
python:3.7
bedevere-bot
removed
the
awaiting merge
label
miss-islington
deleted the
backport-c923c34-3.7
branch
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment