Hi Mandy, Thank you very much for the review. I have added @modules java.instrument, making no other changes. Please, find the updated webrev at http://cr.openjdk.java.net/~akulyakh/8139368_04/
Best regards, Alexander ----- Original Message ----- From: [email protected] To: [email protected] Cc: [email protected], [email protected] Sent: Friday, September 9, 2016 12:28:30 AM GMT +03:00 Iraq Subject: Re: RFR:8139368:-javaagent and -Dcom.sun.management need to add to the initial set of modules to resolve > On Sep 8, 2016, at 6:28 AM, Alexander Kulyakhtin > <[email protected]> wrote: > > Hi, > > Could you, please, review this test-only change (adding a new test). > > CR: https://bugs.openjdk.java.net/browse/JDK-8139368 "-javaagent and > -Dcom.sun.management need to add to the initial set of modules to resolve" > Webrev: http://cr.openjdk.java.net/~akulyakh/8139368_03/ This test needs @modules java.instrument. If it’s run on an image without java.instrument, this will fail. Otherwise looks fine. Mandy
