Skip to content

gh-133374: fix test_python_legacy_windows_stdio#134080

Merged
methane merged 2 commits into
python:mainfrom
methane:fix-test_python_legacy_windows_stdio
May 20, 2025
Merged

gh-133374: fix test_python_legacy_windows_stdio#134080
methane merged 2 commits into
python:mainfrom
methane:fix-test_python_legacy_windows_stdio

Conversation

@methane

@methane methane commented May 16, 2025 •
edited by bedevere-app Bot
Loading

Copy link
Copy Markdown
Member

@methane methane requested review from a team and zooba May 16, 2025 07:16
@methane methane mentioned this pull request May 16, 2025
Merged
@chris-eibl

chris-eibl commented May 16, 2025

Copy link
Copy Markdown
Member

See also #133375.

zooba
zooba reviewed May 16, 2025
View reviewed changes

@zooba zooba 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.

This is closer to being ready than the other PR, IMHO. So up to the two contributors to see who gets there first :)

Comment thread Lib/test/test_cmd_line.py Outdated
Comment thread Lib/test/test_cmd_line.py Outdated
@methane methane changed the title fix test_python_legacy_windows_stdio gh-133374: fix test_python_legacy_windows_stdio May 16, 2025
@bedevere-app bedevere-app Bot mentioned this pull request May 16, 2025
Closed
@methane

methane commented May 16, 2025

Copy link
Copy Markdown
Member Author

Should we use creationflags=subprocess.CREATE_NEW_CONSOLE only when not os.isatty(sys.stdin)?

When a Windows developer runs a test on the console, will a new annoying console appear for a moment? (I cannot run test on Windows Desktop soon.)

@zooba

zooba commented May 19, 2025

Copy link
Copy Markdown
Member

When a Windows developer runs a test on the console, will a new annoying console appear for a moment?

It will, but don't we have a resource for this already?

If the new console doesn't appear, annoying characters get typed over the test results for some similar tests. So I'm pretty sure they're all behind a resource.

@methane methane merged commit 652d693 into python:main May 20, 2025
@serhiy-storchaka serhiy-storchaka added needs backport to 3.13 bugs and security fixes needs backport to 3.14 bugs and security fixes labels May 20, 2025
@miss-islington-app

miss-islington-app Bot commented May 20, 2025

Copy link
Copy Markdown

Thanks @methane for the PR 🌮🎉.. I'm working now to backport this PR to: 3.13.
🐍🍒⛏🤖 I'm not a witch! I'm not a witch!

@miss-islington-app

miss-islington-app Bot commented May 20, 2025

Copy link
Copy Markdown

Thanks @methane for the PR 🌮🎉.. I'm working now to backport this PR to: 3.14.
🐍🍒⛏🤖 I'm not a witch! I'm not a witch!

@miss-islington-app

miss-islington-app Bot commented May 20, 2025

Copy link
Copy Markdown

Sorry, @methane, I could not cleanly backport this to 3.13 due to a conflict.
Please backport using cherry_picker on command line.

cherry_picker 652d6938ef8c42c1c4c180c3f0e257c26c6677da 3.13

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request May 20, 2025
@methane @miss-islington
pythongh-133374: fix test_python_legacy_windows_stdio (pythonGH-134080)
c95962a 
(cherry picked from commit 652d693)

Co-authored-by: Inada Naoki <songofacandy@gmail.com>
@bedevere-app

bedevere-app Bot commented May 20, 2025

Copy link
Copy Markdown

GH-134314 is a backport of this pull request to the 3.14 branch.

@bedevere-app bedevere-app Bot removed the needs backport to 3.14 bugs and security fixes label May 20, 2025
@serhiy-storchaka serhiy-storchaka removed the needs backport to 3.13 bugs and security fixes label May 20, 2025
serhiy-storchaka pushed a commit that referenced this pull request May 20, 2025
@miss-islington @methane
[3.14] gh-133374: fix test_python_legacy_windows_stdio (GH-134080) (G…
90aa13a 
…H-134314)

(cherry picked from commit 652d693)

Co-authored-by: Inada Naoki <songofacandy@gmail.com>
Pranjal095 pushed a commit to Pranjal095/cpython that referenced this pull request Jul 12, 2025
@methane @Pranjal095
pythongh-133374: fix test_python_legacy_windows_stdio (pythonGH-134080)
9259b0a
taegyunkim pushed a commit to taegyunkim/cpython that referenced this pull request Aug 4, 2025
@methane @taegyunkim
pythongh-133374: fix test_python_legacy_windows_stdio (pythonGH-134080)
3fd0e07
@methane methane deleted the fix-test_python_legacy_windows_stdio branch June 26, 2026 06:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@zooba zooba zooba approved these changes

Assignees

@methane methane

Labels

OS-windows skip news tests Tests in the Lib/test dir

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@methane @chris-eibl @zooba @serhiy-storchaka