RFR(S): 8201430: [TESTBUG] Remove script from runtime/6626217

Doerr, Martin martin.doerr at sap.com
Thu Apr 12 12:38:20 UTC 2018

Hi Götz,

I agree with David. Please fix the "!errorMsg.contains("")". Besides that, the change looks good to me as long as the jtreg commands work as expected (which I'm not so familiar with). At least I like your version more than using the script.

Best regards,

-----Original Message-----
From: hotspot-runtime-dev [mailto:hotspot-runtime-dev-bounces at openjdk.java.net] On Behalf Of David Holmes
Sent: Mittwoch, 11. April 2018 23:40
To: Lindenmaier, Goetz <goetz.lindenmaier at sap.com>; hotspot-runtime-dev at openjdk.java.net
Subject: Re: RFR(S): 8201430: [TESTBUG] Remove script from runtime/6626217

Hi Goetz,

This mostly seems reasonable ... though I'm never too sure when jtreg 
might decide to recompile something when processing @run.

In runtime/6626217/bug_21227.java:

  92             if (!errorMsg.contains("")) {

this doesn't look right - aren't you looking for "loader constraint"?


On 11/04/2018 10:58 PM, Lindenmaier, Goetz wrote:
> Hi,
> having looked into this anyways, I removed the script from
> this test.
> Please review:
> http://cr.openjdk.java.net/~goetz/wr18/8201430-removeScript/01
> Best regards,
>    Goetz.

More information about the hotspot-runtime-dev mailing list