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

stdinservice_test: Simplify platform differences #547

Merged
merged 1 commit into from
Dec 16, 2024

Conversation

copybara-service[bot]
Copy link

@copybara-service copybara-service bot commented Dec 10, 2024

stdinservice_test: Simplify platform differences

Python auto-converts newlines into the platform-dependent form, which trips up
our equality check. Make sure that we don't have newlines in the output.

Write through sys.stdout (it works in both Python 2 and Python 3)

@copybara-service copybara-service bot added the exported Exported change from internal source label Dec 10, 2024
@copybara-service copybara-service bot force-pushed the cl/704729437 branch 2 times, most recently from d5e08b7 to c87a46b Compare December 16, 2024 13:10
Python auto-converts newlines into the platform-dependent form, which trips up
our equality check.  Make sure that we don't have newlines in the output.

Write through `sys.stdout` (it works in both Python 2 and Python 3)

PiperOrigin-RevId: 706670816
@copybara-service copybara-service bot merged commit 67f05a9 into master Dec 16, 2024
@copybara-service copybara-service bot deleted the cl/704729437 branch December 16, 2024 13:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
exported Exported change from internal source
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant