RFR: JDK-8193055 ADD_JVM_ARG_IF_OK always fails

Magnus Ihse Bursie magnus.ihse.bursie at oracle.com
Tue Dec 5 10:49:20 UTC 2017


We mistakenly use -XX:-UnlockDiagnosticVMOptions instead of 
-XX:+UnlockDiagnosticVMOptions in a ADD_JVM_ARG_IF_OK call. This means 
that the test will always fail and the arguments will never be added to 
the command line.

Since this has been the case all time along, it's probably not that 
important, but it was added to keep down the logging when using a debug 
build as boot jdk, which has probably not been tested that much either.

We should either fix this or remove the arguments completely.

Bug: https://bugs.openjdk.java.net/browse/JDK-8193055
Patch inline:
diff --git a/make/autoconf/boot-jdk.m4 b/make/autoconf/boot-jdk.m4
--- a/make/autoconf/boot-jdk.m4
+++ b/make/autoconf/boot-jdk.m4
@@ -354,7 +354,7 @@
    AC_MSG_CHECKING([flags for boot jdk java command] )

    # Disable special log output when a debug build is used as Boot JDK...
-  ADD_JVM_ARG_IF_OK([-XX:-PrintVMOptions -XX:-UnlockDiagnosticVMOptions 
-XX:-LogVMOutput],boot_jdk_jvmargs,[$JAVA])
+  ADD_JVM_ARG_IF_OK([-XX:-PrintVMOptions -XX:+UnlockDiagnosticVMOptions 
-XX:-LogVMOutput],boot_jdk_jvmargs,[$JAVA])

    # Force en-US environment
    ADD_JVM_ARG_IF_OK([-Duser.language=en 
-Duser.country=US],boot_jdk_jvmargs,[$JAVA])

/Magnus



More information about the build-dev mailing list