Chris, Misha, George
Thank you for review.
Leonid
On 12.05.2016 14:21, Christian Tornqvist wrote:
Hi Leonid,
Change looks good.
Thanks,
Christian
On May 11, 2016, at 12:36 PM, Leonid Mesnik <[email protected]
<mailto:[email protected]>> wrote:
Hi
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
