Re: RFR: JDK-8170861 : Remove DcmdMBeanPermissionsTest.java from ProblemList

2016-12-19 Thread David Holmes
Looks fine. Thanks, David On 20/12/2016 4:29 PM, Ujwal Vangapally wrote: Please review this small change https://bugs.openjdk.java.net/browse/JDK-8170861 webrev: http://cr.openjdk.java.net/~asapre/sponsorships/Ujwal/JDK-8170861/webrev.00/ Thanks, Ujwal.

RFR: JDK-8170861 : Remove DcmdMBeanPermissionsTest.java from ProblemList

2016-12-19 Thread Ujwal Vangapally
Please review this small change https://bugs.openjdk.java.net/browse/JDK-8170861 webrev: http://cr.openjdk.java.net/~asapre/sponsorships/Ujwal/JDK-8170861/webrev.00/ Thanks, Ujwal.

Re: Review Request JDK-8171468: sun/management/jmxremote/bootstrap/CustomLauncherTest.java fails as lib/$ARCH no longer exists

2016-12-19 Thread Mandy Chung
I did upload the right version a moment ago. Mandy > On Dec 19, 2016, at 1:43 PM, Claes Redestad wrote: > > Hi, > > seems you forgot to remove the code using LIBARCH around line 82 > > /Claes > > On 2016-12-19 22:41, Mandy Chung wrote: >> Webrev: >>

Re: Review Request JDK-8171468: sun/management/jmxremote/bootstrap/CustomLauncherTest.java fails as lib/$ARCH no longer exists

2016-12-19 Thread Claes Redestad
Oops, seems you fixed it, nevermind :-) /Claes On 2016-12-19 22:43, Claes Redestad wrote: Hi, seems you forgot to remove the code using LIBARCH around line 82 /Claes On 2016-12-19 22:41, Mandy Chung wrote: Webrev: http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8171468/webrev.00/ The

Re: Review Request JDK-8171468: sun/management/jmxremote/bootstrap/CustomLauncherTest.java fails as lib/$ARCH no longer exists

2016-12-19 Thread Claes Redestad
Hi, seems you forgot to remove the code using LIBARCH around line 82 /Claes On 2016-12-19 22:41, Mandy Chung wrote: Webrev: http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8171468/webrev.00/ The test is updated to check the new location of shared libraries. lib/$ARCH has been removed.

Re: Review Request JDK-8171468: sun/management/jmxremote/bootstrap/CustomLauncherTest.java fails as lib/$ARCH no longer exists

2016-12-19 Thread Alan Bateman
On 19/12/2016 21:41, Mandy Chung wrote: Webrev: http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8171468/webrev.00/ The test is updated to check the new location of shared libraries. lib/$ARCH has been removed. This looks good. -Alan

Review Request JDK-8171468: sun/management/jmxremote/bootstrap/CustomLauncherTest.java fails as lib/$ARCH no longer exists

2016-12-19 Thread Mandy Chung
Webrev: http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8171468/webrev.00/ The test is updated to check the new location of shared libraries. lib/$ARCH has been removed. Mandy