In NativeCompilation.gmk, the exit value saving for linking is most likely redundant. I managed to remove that for the compile command lines. There I verified that failed compiles still failed the build, without the need for exitvalue saving. I do believe this was relying on the -e switch to bash (I tried both with and without that and pipefail to be sure which was required), but we do have that always on windows.

/Erik

On 2016-01-24 09:56, Magnus Ihse Bursie wrote:
This patch contains a series of small fixes of general applicability that have been created in the build-infra project. These are:

* Add missing llvm library dependency
* Improve compare script for static libraries
* Simplify LogFailures interface, and add support for storing command lines * Hide "Creating library *.lib and object *.exp" text when linking on Windows
* Fix problems with smartjavac when using the build-compare functionality
* When repeating output, truncate it if too long

Bug: https://bugs.openjdk.java.net/browse/JDK-8148120
WebRev: http://cr.openjdk.java.net/~ihse/JDK-8148120-incremental-updates-from-build-infra/webrev.01

/Magnus


Reply via email to