RFR(S): [TESTBUG]runtime/ErrorHandling fails when using jtreg 4.1 b13

[David Holmes]

Hi Igor,

On 25/10/2018 3:11 PM, Igor Ignatyev wrote:
> IIRC, prior jtreg 4.1b13 it was fine to have nonexistent classes in @build actions, so the fix is good regardless of used version of jtreg, but are necessary for jtreg 4.1b13+.

Okay, just wanted to be sure that 8u testing still expects use of 
hotspot testlibrary implementation in hotspot tests.

Though if the @build is actually a no-op is it even necessary to fix it 
rather than just delete it? Sorry I've lost track of where we stand with 
needing to explicitly build the test library in older versions.

Thanks,
David

> -- Igor
>> On Oct 24, 2018, at 10:06 PM, David Holmes <david.holmes at oracle.com> wrote:
>>
>> Hi Vaibhav,
>>
>> cc'ing jdk8u-dev to get some input from the maintainers.
>>
>> I was very confused until I realized this RFR was for 8u! :)
>>
>> Even so, after reading the bug report I'm still quite confused. This has been "broken" since the test was added in January 2016, but only causes a failure if jtreg 4.1b13 is used - is that correct? What is the default version of jtreg being used for 8u today?
>>
>> I'd be happy to review this but given I have no idea how the 8u testing is being handled in general I can't say whether this is the right fix ??
>>
>> Thanks,
>> David
>>
>> On 25/10/2018 2:18 PM, Vaibhav Choudhary wrote:
>>> Hi,
>>> Please review this tiny fix.
>>> Bug : https://bugs.openjdk.java.net/browse/JDK-8151295
>>> Fix: http://cr.openjdk.java.net/~rpatil/8151295/webrev.00/
>>> Desc: In stead of @build jdk.test.lib.* we need to use @build com.oracle.java.testlibrary.*
>>> These two tests are failing in nightly.
>>> Thanks,
>>> Vaibhav
>