X Tutup
The Wayback Machine - https://web.archive.org/web/20221214075420/https://github.com/python/cpython/pull/100154
Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

GH-100133: fix asyncio subprocess losing stderr and stdout output #100154

Open
wants to merge 9 commits into
base: main
Choose a base branch
from
Open

GH-100133: fix asyncio subprocess losing stderr and stdout output #100154

wants to merge 9 commits into from
+18 −3

Conversation

kumaraditya303
Copy link
Contributor

@kumaraditya303 kumaraditya303 commented Dec 10, 2022 •
edited by bedevere-bot

@kumaraditya303 kumaraditya303 added type-bug An unexpected behavior, bug, or error expert-asyncio needs backport to 3.11 labels Dec 10, 2022
@kumaraditya303 kumaraditya303 self-assigned this Dec 10, 2022
@kumaraditya303 kumaraditya303 requested a review from gvanrossum as a code owner Dec 10, 2022
@bedevere-bot bedevere-bot mentioned this pull request Dec 10, 2022
Open
@kumaraditya303 kumaraditya303 changed the title GH-100133: fix asyncio subprocess losing stderr and stdout GH-100133: fix asyncio subprocess losing stderr and stdout output Dec 10, 2022
@kumaraditya303 kumaraditya303 linked an issue Dec 10, 2022 that may be closed by this pull request
asyncio subprocess stdout occasionally lost (3.11.0 → 3.11.1 regression) #100133
Open
@python python deleted a comment from netlify bot Dec 10, 2022
@kumaraditya303 kumaraditya303 added the 🔨 test-with-buildbots Test the PR with the buildbot fleet and report in the status section label Dec 10, 2022
@bedevere-bot bedevere-bot removed the 🔨 test-with-buildbots Test the PR with the buildbot fleet and report in the status section label Dec 10, 2022
@python python deleted a comment from bedevere-bot Dec 10, 2022
@kumaraditya303 kumaraditya303 added the 🔨 test-with-buildbots Test the PR with the buildbot fleet and report in the status section label Dec 10, 2022
@bedevere-bot
Copy link

bedevere-bot commented Dec 10, 2022

🤖 New build scheduled with the buildbot fleet by @kumaraditya303 for commit 3f55f14 🤖

If you want to schedule another build, you need to add the "🔨 test-with-buildbots" label again.

@bedevere-bot bedevere-bot removed the 🔨 test-with-buildbots Test the PR with the buildbot fleet and report in the status section label Dec 10, 2022
@gvanrossum
Copy link
Member

gvanrossum commented Dec 10, 2022

Meta-comment: even though the commits will be squashed, as a reviewer I still appreciate more meaningful commit descriptions. (See also https://xkcd.com/1296/ :-)

I'll review later.

@kumaraditya303
Copy link
Contributor Author

kumaraditya303 commented Dec 11, 2022 •
edited

Ok will do next time, I was in hurry for this one and being not well doesn't help either :(

@gvanrossum
Copy link
Member

gvanrossum commented Dec 12, 2022

Hm, this should probably still be a draft. Also, I'd like to understand what change between 3.11.0 and 3.11.1 caused the regression, so I'm bisecting.

gvanrossum
gvanrossum reviewed Dec 12, 2022
View changes
Copy link
Member

@gvanrossum gvanrossum left a comment

There's some cleanup that needs to happen here.

I would also like to think more about whether the flow is now correct.

Lib/asyncio/base_subprocess.py Outdated Show resolved Hide resolved
Lib/test/test_asyncio/test_subprocess.py Show resolved Hide resolved
Misc/NEWS.d/next/Library/2022-12-10-08-36-07.gh-issue-100133.g-zQlp.rst Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gvanrossum gvanrossum

@1st1 1st1

@asvetlov asvetlov

Assignees

@kumaraditya303 kumaraditya303

Labels
awaiting core review expert-asyncio needs backport to 3.11 type-bug An unexpected behavior, bug, or error
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

asyncio subprocess stdout occasionally lost (3.11.0 → 3.11.1 regression)
3 participants
@kumaraditya303 @bedevere-bot @gvanrossum
X Tutup