Re: [RFR] (S) 8140189: [TESTBUG] Get rid of "@library /../../test/lib" in jtreg tests

2015-10-29 Thread Staffan Larsen
Looks good (still)! Thanks, /Staffan > On 28 okt. 2015, at 06:42, Chris Plummer wrote: > > Hello, > > I've fixed the new jvmci tests. New webrevs found here: > > http://cr.openjdk.java.net/~cjplummer/8140189/webrev.01/webrev.hotspot >

Re: [RFR] (S) 8140189: [TESTBUG] Get rid of "@library /../../test/lib" in jtreg tests

2015-10-26 Thread Chris Plummer
I just pulled the latest hs-rt, and got about 30 new jvmci tests that are using "/../../test/lib". I can fix them with this push, or file a separate bug or send out a fix after I do this push. If I fix with this push, do you want another review? I'll test with jprt and run the jvmci tests

Re: [RFR] (S) 8140189: [TESTBUG] Get rid of "@library /../../test/lib" in jtreg tests

2015-10-23 Thread Staffan Larsen
Looks good! Thanks for doing this. /Staffan > On 23 okt. 2015, at 07:54, Chris Plummer wrote: > > Hello, > > Please review the following fix for 8140189: > > http://cr.openjdk.java.net/~cjplummer/8140189/webrev.00/webrev.hotspot >

[RFR] (S) 8140189: [TESTBUG] Get rid of "@library /../../test/lib" in jtreg tests

2015-10-22 Thread Chris Plummer
Hello, Please review the following fix for 8140189: http://cr.openjdk.java.net/~cjplummer/8140189/webrev.00/webrev.hotspot http://cr.openjdk.java.net/~cjplummer/8140189/webrev.00/webrev.jdk https://bugs.openjdk.java.net/browse/JDK-8140189 Please also see the following CR, which has much more