-
-
Notifications
You must be signed in to change notification settings - Fork 265
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
precompilation: capture stdout as well as stderr #3573
precompilation: capture stdout as well as stderr #3573
Conversation
Another question is whether we should just do this always and not just code load |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is strongly forbidden for the same object to be used asynchronously in two difference places.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be using a Pipe()
. There also should not be a close(iob)
call, ever, since that corrupts the stream.
0e09878
to
b850e23
Compare
(cherry picked from commit 0a4fb55)
Fixes #3548
@vtjnash in particular is the use of a single BufferStream
iob
ok here or should it use two buffers?