ping: RE: RFR(S): 7901780: jtreg must set LIBPATH on aix/os400

[Jonathan Gibbons]

Goetz,

It was easy enough to edit the filenames in the patch, to account for 
the recent jtreg package reorg, without otherwise changing the content 
of the patch. I have therefore sponsored the patch on your behalf and 
pushed the change.

One other bureaucratic change was necessary.  This mail thread, and the 
changeset comment, was incorrectly using the wrong issue number. Not 
surprisingly, it turned out to be the issue for an earlier contribution 
of yours.  Luckily, jcheck caught the mistake, and the summary in the 
mail subject line and the changeset comment was enough to identify the 
correct number to use.

For the record:

7901696: os.simpleArch is x64 for linux-ppc64le in @require context
7901780: jtreg must set LIBPATH on aix/os400

-- Jon

On 10/27/2016 08:57 AM, Jonathan Gibbons wrote:
> Goetz,
>
> Sorry for the delay. The patch looks OK, but the jtreg classes have 
> been shuffled since you created the patch.  I can sponsor the change 
> for you and take the shuffle into account, or you can submit a new 
> webrev/patch, your choice.
>
> -- Jon
>
>
> On 10/27/16 3:37 AM, Lindenmaier, Goetz wrote:
>> Hi,
>>
>> could someone please have a look at this change?
>>
>> Thanks,
>>    Goetz
>>
>>> -----Original Message-----
>>> From: Lindenmaier, Goetz
>>> Sent: Dienstag, 6. September 2016 11:15
>>> To: code-tools-dev at openjdk.java.net
>>> Subject: RFR(S): 7901696: jtreg must set LIBPATH on aix/os400
>>>
>>> Hi
>>>
>>>
>>>
>>> Jtreg sets LD_LIBRARY_PATH on aix and os400, where it must use
>>>
>>> LIBPATH.
>>>
>>>
>>>
>>> Please review this change. I please need a sponsor (or can I push 
>>> myself?)
>>>
>>> http://cr.openjdk.java.net/~goetz/wr16/7901696-jtreg_LIBPATH/webrev.01/
>>>
>>>
>>>
>>> Best regards,
>>>
>>>    Goetz.
>