Hi
<https://bugs.openjdk.java.net/browse/JDK-8156777>
Could you please review this extra small fix which add @requires to the single test. This test is not compatible with minimal VM even it doesn't require any specific modules outside compact2. So I've just added corresponding @requires.

Bug: https://bugs.openjdk.java.net/browse/JDK-8156777

Tested by manual jtreg execution with -minimal/-client/default vm options.

Fix inlined:

--- a/test/testlibrary_tests/SimpleClassFileLoadHookTest.java   Wed May 11 
02:32:14 2016 -0400
+++ b/test/testlibrary_tests/SimpleClassFileLoadHookTest.java   Wed May 11 
19:24:34 2016 +0300
@@ -24,7 +24,7 @@
 /*
  * @test
  * @library /testlibrary
- *
+ * @requires vm.flavor != "minimal"
  * @run main/othervm/native -agentlib:SimpleClassFileLoadHook=Foo,XXX,YYY
  *      SimpleClassFileLoadHookTest
  */


Leonid

Reply via email to