When using the RC tool on Windows, we also run CL with the same
arguments to generate dependencies on included files. In some cases,
extra include directories are provided in the RC_FLAGS, using the flag
-i (lower case). This flag works for the RC tool, but not for CL, which
requires -I (upper case). The upper case version works for RC as well so
we should switch to using upper case.
Another problem uncovered by this is that NativeCompilation.gmk isn't
handling this failure well. It calls CL through the ExecuteWithLog
macro. This macro will copy the log to the failure-logs dir if the
command returns non zero. Because of the above problem, the command
returns non zero. NativeCompilation.gmk ignores this so the build
continues. If the build then fails for any other reason, the failure
report will find these failure logs and fool the user into thinking this
was the cause of the build failure.
I propose to fix this by changing NativeCompilation.gmk to not ignore
the return code of CL and let the output of CL through, but filtered the
same way as when we do normal compilation. Then the build will actually
fail if CL fails here so there is no discrepancy between failure-logs
and the actual failure. I also removed the piping to a new file since
ExecuteWithLog already pipes output to a file so we can just use that.
To make this work, we must also ensure that no bad arguments reach CL
here. This is done by filtering known bad arguments (currently only
-l0x409 which I chose to filter with +l%, note that this is lower case l
and not upper case i), and of course changing -i to -I in all places.
Bug: https://bugs.openjdk.java.net/browse/JDK-8156837
Webrev: http://cr.openjdk.java.net/~erikj/8156837/webrev.01/
/Erik
