Thank you, Mandy On 10/4/17 4:15 PM, mandy chung wrote:
+1
Looks like JDK regression tests were not run before pushing JDK-8182701?
Yes, only hotspot jtreg tests were run unfortunately before the push. We do run jdk_lang regularly in tier5 Nightly testing. Thanks, Vladimir
Mandy
On 10/4/17 4:12 PM, Vladimir Kozlov wrote:
https://bugs.openjdk.java.net/browse/JDK-8188776
8182701 added exports for jdk.vm.ci.runtime package [1] but did not add new exception in the test.
Added missing exception in JdkQualifiedExportTest.java test:
--- a/test/jdk/jdk/modules/etc/JdkQualifiedExportTest.java +++ b/test/jdk/jdk/modules/etc/JdkQualifiedExportTest.java @@ -70,6 +70,7 @@
static Set<String> KNOWN_EXCEPTIONS = Set.of("jdk.internal.vm.ci/jdk.vm.ci.services", + "jdk.internal.vm.ci/jdk.vm.ci.runtime", "jdk.jsobject/jdk.internal.netscape.javascript.spi");
static void checkExports(ModuleDescriptor md) {
Verified with this test.
Thanks, Vladimir
[1] http://hg.openjdk.java.net/jdk10/hs/rev/8b2054b7d02c#l3.1