tree a40bdf96819c289a4aa24e077affd22380b61393
parent 543f6e666d964ae091bb911a9aef687b980ce7ba
author janakr <janakr@google.com> 1605919809 -0800
committer Copybara-Service <copybara-worker@google.com> 1605919867 -0800

Remove ostentatious interleaving of action output and progress messages.

I traced the use of FileOutErr to only be in SkyframeActionExecutor. The other event types generally wouldn't be expected to have stdout/stderr in any case.

Detailed changes:

* Change writeToStream to never show the progress bar, the one place it was asked to show the progress bar now does so explicitly.
* Move the display of any stdout/stderr to inside the switch cases that can have stdout/stderr in the first place, specifically before we redraw the progress bar.
* Don't flush the terminal before printing stdout/stderr, I don't see a reason to do that.
* Assert that no other events have stdout/stderr in the first place.

PiperOrigin-RevId: 343589378