craftr-build / craftr-build-4.x

Frontend for the Craftr build framework.
https://craftr-build.github.io/craftr/
Other
60 stars 14 forks source link

Don't patch sys.stdin, stdout and stderr #102

Closed NiklasRosenstein closed 7 years ago

NiklasRosenstein commented 8 years ago

Craftr RTS patches sys.stdin, sys.stdout and sys.stderr in order to capture output from the tasks it executes in separate threads.

# xxx: Is there any other way than hard-patching the sys module?
thread_stdin = sys.stdin = ThreadIO(sys.stdin)
thread_stdout = sys.stdout = ThreadIO(sys.stdout)
thread_stderr = sys.stderr = ThreadIO(sys.stderr)

It would be nice if we found a way to avoid that.

NiklasRosenstein commented 7 years ago

Closing in favour of #163