RE: RFR(XS): 8217438: Adapt tools//launcher/Test7029048.java for AIX

2019-01-22 Thread Lindenmaier, Goetz
Hi Arno, 

thanks for looking at the change!

Best regards,
  Goetz.

> -Original Message-
> From: Zeller, Arno
> Sent: Dienstag, 22. Januar 2019 14:48
> To: Lindenmaier, Goetz 
> Cc: core-libs-dev@openjdk.java.net
> Subject: RE: RFR(XS): 8217438: Adapt tools//launcher/Test7029048.java for
> AIX
> 
> Hi Goetz,
> 
> looks good to me and should fix the issue.
> I just want to point out that I am not a Reviewer.
> 
> Best regards,
> Arno
> 
> >-Original Message-
> >From: core-libs-dev  On Behalf Of
> >Lindenmaier, Goetz
> >Sent: Montag, 21. Januar 2019 11:01
> >To: core-libs-dev@openjdk.java.net
> >Subject: [CAUTION] RFR(XS): 8217438: Adapt
> >tools//launcher/Test7029048.java for AIX
> >
> >Hi,
> >
> >please review the following small test fix:
> >http://cr.openjdk.java.net/~goetz/wr19/8217438-aix_launcherTest/01/
> >
> >The Aix launcher always sets the path to the jdk libraries because
> >Aix does not support $ORIGIN to find libjvm.so.
> >So some of the test cases here fail. Skip them.
> >
> >Best regards,
> >  Goetz.



RE: RFR(XS): 8217438: Adapt tools//launcher/Test7029048.java for AIX

2019-01-22 Thread Zeller, Arno
Hi Goetz,

looks good to me and should fix the issue.
I just want to point out that I am not a Reviewer.

Best regards,
Arno

>-Original Message-
>From: core-libs-dev  On Behalf Of
>Lindenmaier, Goetz
>Sent: Montag, 21. Januar 2019 11:01
>To: core-libs-dev@openjdk.java.net
>Subject: [CAUTION] RFR(XS): 8217438: Adapt
>tools//launcher/Test7029048.java for AIX
>
>Hi,
>
>please review the following small test fix:
>http://cr.openjdk.java.net/~goetz/wr19/8217438-aix_launcherTest/01/
>
>The Aix launcher always sets the path to the jdk libraries because
>Aix does not support $ORIGIN to find libjvm.so.
>So some of the test cases here fail. Skip them.
>
>Best regards,
>  Goetz.



RE: RFR(XS): 8217438: Adapt tools//launcher/Test7029048.java for AIX

2019-01-22 Thread Lindenmaier, Goetz
Hi Christoph, 

thanks for your review. I'll update AIX to Aix before pushing.

Best regards,
  Goetz.

> -Original Message-
> From: Langer, Christoph
> Sent: Montag, 21. Januar 2019 11:43
> To: Lindenmaier, Goetz 
> Cc: core-libs-dev@openjdk.java.net
> Subject: RE: RFR(XS): 8217438: Adapt tools//launcher/Test7029048.java for
> AIX
> 
> Hi Goetz,
> 
> looks good to me. I think it should rather be "AIX" than "Aix" in the output 
> text
> - but it's a developer trace only, so probably nobody cares. 
> 
> Best regards
> Christoph
> 
> > -Original Message-
> > From: core-libs-dev  On Behalf
> > Of Lindenmaier, Goetz
> > Sent: Montag, 21. Januar 2019 11:01
> > To: core-libs-dev@openjdk.java.net
> > Subject: [CAUTION] RFR(XS): 8217438: Adapt
> > tools//launcher/Test7029048.java for AIX
> >
> > Hi,
> >
> > please review the following small test fix:
> > http://cr.openjdk.java.net/~goetz/wr19/8217438-aix_launcherTest/01/
> >
> > The Aix launcher always sets the path to the jdk libraries because
> > Aix does not support $ORIGIN to find libjvm.so.
> > So some of the test cases here fail. Skip them.
> >
> > Best regards,
> >   Goetz.



RE: RFR(XS): 8217438: Adapt tools//launcher/Test7029048.java for AIX

2019-01-21 Thread Langer, Christoph
Hi Goetz,

looks good to me. I think it should rather be "AIX" than "Aix" in the output 
text - but it's a developer trace only, so probably nobody cares. 

Best regards
Christoph

> -Original Message-
> From: core-libs-dev  On Behalf
> Of Lindenmaier, Goetz
> Sent: Montag, 21. Januar 2019 11:01
> To: core-libs-dev@openjdk.java.net
> Subject: [CAUTION] RFR(XS): 8217438: Adapt
> tools//launcher/Test7029048.java for AIX
> 
> Hi,
> 
> please review the following small test fix:
> http://cr.openjdk.java.net/~goetz/wr19/8217438-aix_launcherTest/01/
> 
> The Aix launcher always sets the path to the jdk libraries because
> Aix does not support $ORIGIN to find libjvm.so.
> So some of the test cases here fail. Skip them.
> 
> Best regards,
>   Goetz.