Skip to content

gh-109276, gh-109508: Fix libregrtest stdout #109903

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

Merged
merged 1 commit into from
Sep 26, 2023

Conversation

vstinner
Copy link
Member

@vstinner vstinner commented Sep 26, 2023

Remove replace_stdout(): call sys.stdout.reconfigure() instead of set the error handler to backslashreplace.

display_header() logs an empty line and flush stdout.

Remove encoding workaround in display_header() since stdout error handler is now set to backslashreplace earlier.

Remove replace_stdout(): call sys.stdout.reconfigure() instead of set
the error handler to backslashreplace.

display_header() logs an empty line and flush stdout.

Remove encoding workaround in display_header() since stdout error
handler is now set to backslashreplace earlier.
@vstinner vstinner merged commit b1e4f6e into python:main Sep 26, 2023
@vstinner vstinner deleted the regrtest_stdout branch September 26, 2023 19:34
csm10495 pushed a commit to csm10495/cpython that referenced this pull request Sep 28, 2023
Remove replace_stdout(): call sys.stdout.reconfigure() instead of set
the error handler to backslashreplace.

display_header() logs an empty line and flush stdout.

Remove encoding workaround in display_header() since stdout error
handler is now set to backslashreplace earlier.
Glyphack pushed a commit to Glyphack/cpython that referenced this pull request Sep 2, 2024
Remove replace_stdout(): call sys.stdout.reconfigure() instead of set
the error handler to backslashreplace.

display_header() logs an empty line and flush stdout.

Remove encoding workaround in display_header() since stdout error
handler is now set to backslashreplace earlier.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant