RFR(trivial): 8223054: [TESTBUG] Put graalJarsCP before existing classpath in GraalUnitTestLauncher
Pengfei Li (Arm Technology China)
Pengfei.Li at arm.com
Sun May 5 01:32:34 UTC 2019
Hi,
Please help review this trivial change on GraalUnitTestLauncher.
Webrev: http://cr.openjdk.java.net/~pli/rfr/8223054/webrev.00/
JBS: https://bugs.openjdk.java.net/browse/JDK-8223054
Current graal unit test in jtreg requires junit-4.12.jar as a dependency. In GraalUnitTestLauncher.java, we put the path of this file into graalJarsCP and concat it with existing classpath. But existing classpath may contain another version of junit with which the jtreg tool is built. (According to OpenJDK "Building jtreg" webpage[1], the recommended version of Junit to build jtreg is junit-4.10).
In this patch, graalJarsCP is put before existing classpath returned by System.getProperty() when generating the new classpath string to avoid incompatibility issues. Jteg graal unit test cases passed after this change.
[1] https://openjdk.java.net/jtreg/build.html
--
Thanks,
Pengfei
More information about the hotspot-runtime-dev
mailing list