JDK-8152993 : All launcher tests fail to launch in non-jigsaw legacy mode
  • Type: Bug
  • Component: javafx
  • Sub-Component: build
  • Affected Version: 9
  • Priority: P2
  • Status: Resolved
  • Resolution: Fixed
  • Submitted: 2016-03-29
  • Updated: 2016-03-29
  • Resolved: 2016-03-29
The Version table provides details related to the release that this issue/RFE will be addressed.

Unresolved : Release in which this issue/RFE will be addressed.
Resolved: Release in which this issue/RFE has been resolved.
Fixed : Release in which this issue/RFE has been fixed. The release containing this fix may be available for download as an Early Access Release or a General Availability Release.

To download the current JDK release, click here.
JDK 9
9Fixed
Related Reports
Relates :  
JDK-8150307 - Some tests fail when run on JDK 9 due to use of -Djava.ext.dirs
Relates :  
JDK-8147399 - Enable systemTests in jake
Description
To reproduce this, run the following in non-jigsaw "legacy" mode.

$ unset JIGSAW_HOME
$ gradle --info -PFULL_TEST=true :systemTests:test

The following tests fail to run:

MainLauncherTest
SandboxAppTest
ShutdownHookTest

The error is the same for all of them:

Error: Could not create the Java Virtual Machine.
-Djava.ext.dirs=/localhome/kcr/javafx/9-kcr/jfx/rt/build/sdk/lib is not supported.  Use -classpath instead.

This issue was fixed in 9-dev with JDK-8150307, so either the bug was reintroduced or the fix was lost when merging FX 9 into the 9-jake sandbox.
Comments
http://hg.openjdk.java.net/openjfx/9-dev/rt/rev/7ba1c09a75c1
29-03-2016
+1
29-03-2016
Simple fix (above) to reapply the fix for JDK-8150307
29-03-2016
diff --git a/tests/system/src/test/java/test/util/Util.java b/tests/system/src/test/java/test/util/Util.java --- a/tests/system/src/test/java/test/util/Util.java +++ b/tests/system/src/test/java/test/util/Util.java @@ -204,7 +204,7 @@ } else { String jfxdir = getJfxrtDir(classpath); Assert.assertNotNull("failed to find jfxdir",jfxdir); - cmd.add("-Djava.ext.dirs=" + jfxdir); + cmd.add("-Xbootclasspath/a:" + jfxdir + "/" + "jfxrt.jar"); } // This is a "minimum" set, rather than the full @addExports
29-03-2016
Actually, no, this is not a merge error. The fix for JDK-8147399 reintroduced the java.ext.dirs bug -- I missed it during my code review.
29-03-2016
Looking at the following, it seems this was a merge error: http://cr.openjdk.java.net/~kcr/8092093/webrev.01/rt/tests/system/src/test/java/test/util/Util.java.udiff.html
29-03-2016