Read about our upcoming Code of Conduct on this issue

  • Manuel Jacob's avatar
    procutil: back out 8403cc54bc83 (make ....procutil.stderr unbuffered) · fa270dcbdb55
    Manuel Jacob authored
    Changeset 8403cc54bc83 introduced code that opens a second file object
    referring to the stderr file descriptor. This broke tests on Windows. The
    reason is that on Windows, sys.stderr is buffered and procutil.stderr closed
    the file descriptor when it got garbage collected before sys.stderr had the
    chance to flush buffered data.
    
    `procutil.stdout` had the same problem for a long time, but we didn’t realize,
    as in CI test runs, stdout is not a TTY and in this case no second file object
    is opened.
    fa270dcbdb55
procutil.py 20.8 KB