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

Lindenmaier, Goetz goetz.lindenmaier at sap.com
Fri Oct 28 06:36:43 UTC 2016


Hi Jon, 

thanks for sponsoring, the change! Yes, I actually messed up the bug-ids.
Sorry I wasn't fast enough to do these fixes myself, but I'm on CEST 
schedule.  

Best regards,
  Goetz. 

> -----Original Message-----
> From: code-tools-dev [mailto:code-tools-dev-bounces at openjdk.java.net]
> On Behalf Of Jonathan Gibbons
> Sent: Donnerstag, 27. Oktober 2016 23:24
> To: code-tools-dev at openjdk.java.net
> Subject: Re: ping: RE: RFR(S): 7901780: jtreg must set LIBPATH on aix/os400
> 
> 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.
> >



More information about the code-tools-dev mailing list