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.
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.
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:
>>
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
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.
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
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