Hi Matthias, 

I didin't mean to do any changes for other platforms, so
I reverted the part you pointed out:
http://cr.openjdk.java.net/~goetz/wr18/8194869-jtregAix/webrev.02/

But scanning through all the scripts: 
Sometimes PS on Cygwin is ';' and sometimes it is ':'.  Strange ...

Best regards,
  Goetz.

> -----Original Message-----
> From: Baesken, Matthias
> Sent: Montag, 15. Januar 2018 09:03
> To: Lindenmaier, Goetz <goetz.lindenma...@sap.com>; core-libs-
> d...@openjdk.java.net
> Subject: RE: RFR(S): 8194869: [TESTBUG][aix, s390] Adapt tests to platforms.
> 
> Hi Goetz, change looks good except
> 
> test/jdk/sun/security/tools/keytool/i18n.sh
> 
> where the Cygwin PS setting has been changed from ; to :   (probably this
> was not intended) :
> 
>      PS=":"
>      FS="/"
>      ;;
> -  CYGWIN* )
> -    NULL=/dev/null
> -    PS=";"
> 
> 
> (not a Reviewer however).
> 
> Best regards, Matthias
> 
> 
> 
> > -----Original Message-----
> > From: Lindenmaier, Goetz
> > Sent: Mittwoch, 10. Januar 2018 09:23
> > To: core-libs-dev@openjdk.java.net
> > Cc: Baesken, Matthias <matthias.baes...@sap.com>
> > Subject: RFR(S): 8194869: [TESTBUG][aix, s390] Adapt tests to platforms.
> >
> > Hi,
> >
> > this change adapts a row of jtreg tests to AIX.
> > Shell tests often switch over all OSes, here the case for AIX was
> > missing. PKCSTest needs to know where the libs are on s390.
> >
> > Please review.
> > http://cr.openjdk.java.net/~goetz/wr18/8194869-jtregAix/webrev/
> >
> > Best regards,
> >   Goetz.

Reply via email to