On Tue, 11 Jun 2013, Alexander Bokovoy wrote:
On Mon, 10 Jun 2013, Petr Viktorin wrote:
This is related to testing but can be pushed independently. It will make testing master much more pleasant.

sys.stdout is buffered by default if redirected to a file.
This may cause automated installation to appear hung.
Flush the stream so that messages are written immediately.
Despite the fact that we have abstracted out output_fd in service class,
in existing code it is only used against sys.stdout.

Just in case it would be used against different output_fd, could you please
make flush() conditional under output_fd.isatty() condition?
On further thinking this conditioning is not really needed. So scratch
ACK for the original patch.

/ Alexander Bokovoy

Freeipa-devel mailing list

Reply via email to