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

flushing stdout and stderr causes BrokenPipeError #60

Closed
goodmami opened this issue Jan 31, 2020 · 1 comment
Closed

flushing stdout and stderr causes BrokenPipeError #60

goodmami opened this issue Jan 31, 2020 · 1 comment
Labels
bug Something isn't working

Comments

@goodmami
Copy link
Owner

In 8bd5e99 I started flushing stdout and stderr to synchronize warnings from --check with graphs on stdout, but it turns out this breaks the pipe. See https://stackoverflow.com/questions/26692284/how-to-prevent-brokenpipeerror-when-doing-a-flush-in-python

I can catch the errors and close stderr as described in https://stackoverflow.com/a/26738736/1441112, or do something else, like put the graph number or identifier on stderr before the error messages and not flush at all.

@goodmami goodmami added the bug Something isn't working label Jan 31, 2020
@goodmami
Copy link
Owner Author

goodmami commented Feb 3, 2020

I'm having trouble reproducing now, and besides #65 will make this issue mostly irrelevant as the errors can be put in metadata and not on stderr, which is probably more useful anyway.

This bug can still be closed when the flush calls are removed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant