1039832 - The code in cl.py to capture the showIncludes output should look at stdout

Status: NEW

(Firefox Build System :: General, defect)

Description

[(no longer active)]

The documentation <http://msdn.microsoft.com/en-us/library/hdkef6tk.aspx> might make you believe that the output goes to stderr, but that is a lie.  cl.py seems to intercept both streams, which makes it get lines such as:

Note: including file:  C:\Program Files (x86)\Windows Kits\8.0\include\shc:/moz/src/image/decoders/icon/win/nsIconChannel.cpp(72,1) :  error: static_assert failed "Reference-counted class nsIconChannel should not have a public destructor. Try to

which completely confuses cl.py and results in the error message to go into the .deps file, and also removes it from stdout.

In other words, this bug can eat your error messages, and also break incremental builds.

Comment 1

[(not currently active) Ted Mielczarek]

Same root cause as bug 798300, FWIW. Stupid Windows buffering. :-(

Comment 2

[Mike Hommey [:glandium]]

The fun thing is that the -showIncludes output *can* end up on stderr under some circumstances.

Comment 3

[(no longer active)]

(In reply to comment #2)
> The fun thing is that the -showIncludes output *can* end up on stderr under
> some circumstances.

What are those circumstances?