[8u-dev] Request for review: JDK-8151295: runtime/ErrorHandling fails when using jtreg 4.1 b13

Chris Plummer chris.plummer at oracle.com
Thu Apr 21 19:51:42 UTC 2016


See the following CR for an explanation of the below use of 
external.lib.roots:

JDK-8140189: [TESTBUG] Get rid of "@library /../../test/lib" in jtreg tests
https://bugs.openjdk.java.net/browse/JDK-8140189

Your use case is a bit different, and I don't see how the JDK9 setting 
of external.lib.roots would resolve it. However, I also don't see how 
this test currently is working in JDK9 unless jtreg treats <ws> as an 
implied external.lib.roots, in which case this fix would not be needed 
in 8u either. Maybe John can shed some light on this.

BTW, com.oracle.java.testlibrary no longer exists in JDK9.

cheers,

Chris

On 4/21/16 9:14 AM, Martin Buchholz wrote:
> How does jdk9 work then?
> Answering my own question, I think it's:
>
> # Path to libraries in the topmost test directory. This is needed so @library
> # does not need ../../ notation to reach them
> external.lib.roots = ../../
>
> On Thu, Apr 21, 2016 at 6:58 AM, Alexander Vorobyev
> <alexander.vorobyev at oracle.com> wrote:
>> Hi All,
>>
>> I'd like review for JDK-8151295
>> (https://bugs.openjdk.java.net/browse/JDK-8151295)
>>
>> Looks like new version of jtreg by default does not allow to use libraries
>> from the outside of test suite. So I guess jdk.test.lib.* in affected tests
>> should be changed to com.oracle.java.testlibrary.* which is available in the
>> hotspot/test testsuite. At least until it renamed to jdk.testlibrary (like
>> here https://bugs.openjdk.java.net/browse/JDK-8067013).
>>
>>
>>
>> Here is webrev:
>> http://cr.openjdk.java.net/~avorobye/8151295/
>>
>>
>> Thanks,
>> Alexander
>>
>>
>>



More information about the jdk8u-dev mailing list