Hi Leonid, Change looks good.
Thanks, Christian > On May 11, 2016, at 12:36 PM, Leonid Mesnik <[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
