RFR: JDK-8171859: Configure check for modular boot jdk needs to be updated
Erik Joelsson
erik.joelsson at oracle.com
Thu Dec 22 09:39:45 UTC 2016
Hello,
Configure has a check to see if the boot jdk is module aware/enabled.
Like many others, the build needs to adapt its behavior when using such
a boot jdk. The current check is no longer compatible with the latest
JDK 9 builds:
java -version --patch-module foo=bar
WARNING: Unknown module: foo specified in --patch-module
I propose a simple adjustment, to instead specify java.base as the
module to patch. I also made it explicit that the directory to patch
from does not exist.
I'm open to suggestions on better tests though, but would prefer if they
can be as simple as checking for a command line argument.
Bug: https://bugs.openjdk.java.net/browse/JDK-8171859
Patch:
diff -r 84c58139cbd3 common/autoconf/boot-jdk.m4
--- a/common/autoconf/boot-jdk.m4
+++ b/common/autoconf/boot-jdk.m4
@@ -305,7 +305,9 @@
BOOT_JDK_SOURCETARGET="-source 8 -target 8"
AC_SUBST(BOOT_JDK_SOURCETARGET)
- ADD_JVM_ARG_IF_OK([--patch-module foo=bar], dummy, [$JAVA])
+ # Use a non existing directory as the directory will be scanned otherwise
+ # and can potentially take a lot of time.
+ ADD_JVM_ARG_IF_OK([--patch-module
java.base=SOME_DIR_THAT_DOES_NOT_EXIST], dummy, [$JAVA])
AC_MSG_CHECKING([if Boot JDK supports modules])
if test "x$JVM_ARG_OK" = "xtrue"; then
AC_MSG_RESULT([yes])
/Erik
More information about the build-dev
mailing list