Hi , I don't think the check would do any harm here, but I removed it anyways. No big deal.
Thanks for reviewing! Best regards, Goetz. > -----Original Message----- > From: David Holmes [mailto:david.hol...@oracle.com] > Sent: Donnerstag, 18. Januar 2018 02:18 > To: Volker Simonis <volker.simo...@gmail.com> > Cc: Baesken, Matthias <matthias.baes...@sap.com>; Lindenmaier, Goetz > <goetz.lindenma...@sap.com>; core-libs-dev@openjdk.java.net > Subject: Re: RFR(S): 8194869: [TESTBUG][aix, s390] Adapt tests to platforms. > > Hi Volker, > > On 18/01/2018 9:21 AM, Volker Simonis wrote: > > > > David Holmes <david.hol...@oracle.com > <mailto:david.hol...@oracle.com>> > > schrieb am Mi. 17. Jan. 2018 um 22:45: > > > > Hi Goetz, > > > > On 17/01/2018 10:42 PM, Lindenmaier, Goetz wrote: > > > Hi David, > > > > > > thanks for looking at the change. > > > > > > We actually run the aix VM also on OS400. > > > And as OS.java says that it's a " Stripped down version of jtreg > > > internal class com.sun.javatest.regtest.config.OS" > > > I not only grabbed the AIX case from there, but also the OS400 one. > > > I don't think this will do any harm. > > > > Can't speak to what platforms jtreg runs on but OpenJDK doesn't support > > OS/400, so it shouldn't be listed. Sorry. > > > > > > Hi David, > > > > you’re right that OpenJDK doesn’t run natively on OS/400, but OS/400 > > provides an emulation layer called PASE [1] which allows to run native > > AIX application on OS/400. The change proposed by Goetz (which is > > already in JTreg since quite some time) is required to test the OpenJDK > > AIX Port on OS/400 in PASE mode. > > Sorry but I don't see how that one occurrence in > > test/failure_handler/src/share/classes/jdk/test/failurehandler/jtreg/OS.java > > allows for testing on OS/400 when none of the actual tests that examine > the OS have a case for "os400" ?? (Wouldn't it be better to pretend to > be AIX in that case?) > > David > ----- > > > Regards, > > Volker > > > > [1] > > > https://www.ibm.com/support/knowledgecenter/en/ssw_i5_54/rzalf/rzalfi > ntro.htm > > > > > > > > Thanks, > > David > > > > > I'll post it on awt-dev, too. > > > > > > Best regards, > > > Goetz. > > > > > >> -----Original Message----- > > >> From: David Holmes [mailto:david.hol...@oracle.com > > <mailto:david.hol...@oracle.com>] > > >> Sent: Mittwoch, 17. Januar 2018 12:55 > > >> To: Lindenmaier, Goetz <goetz.lindenma...@sap.com > > <mailto:goetz.lindenma...@sap.com>>; Volker Simonis > > >> <volker.simo...@gmail.com <mailto:volker.simo...@gmail.com>> > > >> Cc: core-libs-dev@openjdk.java.net > > <mailto:core-libs-dev@openjdk.java.net>; Baesken, Matthias > > >> <matthias.baes...@sap.com <mailto:matthias.baes...@sap.com>> > > >> Subject: Re: RFR(S): 8194869: [TESTBUG][aix, s390] Adapt tests > > to platforms. > > >> > > >> Hi Goetz, > > >> > > >> > > > test/failure_handler/src/share/classes/jdk/test/failurehandler/jtreg/OS.java > > >> > > >> I thought this was for AIX recognition not OS/400 ? ;-) > > >> > > >> Otherwise the AIX changes seem okay. Thought AWT tests should be > > >> reviewed on awt-dev. > > >> > > >> Cheers, > > >> David > > >> > > >> On 17/01/2018 9:39 PM, Lindenmaier, Goetz wrote: > > >>> Hi, > > >>> > > >>> Volker, thanks for your review. Could someone else please have a > > >>> quick look at this simple fix? > > >>> > > >>> Updated webrev with fixed BadPluginConfigurationTest: > > >>> http://cr.openjdk.java.net/~goetz/wr18/8194869- > jtregAix/webrev.03/ > > >>> > > >>> Best regards, > > >>> Goetz. > > >>> > > >>>> -----Original Message----- > > >>>> From: Volker Simonis [mailto:volker.simo...@gmail.com > > <mailto:volker.simo...@gmail.com>] > > >>>> Sent: Dienstag, 16. Januar 2018 19:28 > > >>>> To: Lindenmaier, Goetz <goetz.lindenma...@sap.com > > <mailto:goetz.lindenma...@sap.com>> > > >>>> Cc: Baesken, Matthias <matthias.baes...@sap.com > > <mailto:matthias.baes...@sap.com>>; core-libs- > > >>>> d...@openjdk.java.net <mailto:d...@openjdk.java.net> > > >>>> Subject: Re: RFR(S): 8194869: [TESTBUG][aix, s390] Adapt tests to > > >> platforms. > > >>>> > > >>>> Hi Goetz, > > >>>> > > >>>> you change looks good. Just one minor nit (no need for a new > > webrev if > > >>>> you agree with my fix :) > > >>>> > > >>>> In BadPluginConfigurationTest.sh you first check if 'mktemp' > > exists as > > >>>> file. This is a useless check because there won't be a file > > "mktemp" > > >>>> in the current exectution directory of the shell script, even > > if the > > >>>> "mktemp" exists domehwere in the PATH. To check for the > > existence of a > > >>>> "mktemp" executable you can use the "type" command: > > >>>> > > >>>> type mktempp> /dev/null 2>&1 > > >>>> > > >>>> if ! [ $? -ne 0 ] ; then > > >>>> MKTEMP="/opt/freeware/bin/mktemp" > > >>>> fi > > >>>> > > >>>> Thanks for finally fixing this, > > >>>> Volker > > >>>> > > >>>> > > >>>> On Mon, Jan 15, 2018 at 12:12 PM, Lindenmaier, Goetz > > >>>> <goetz.lindenma...@sap.com > <mailto:goetz.lindenma...@sap.com>> > > wrote: > > >>>>> 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 > > <mailto:goetz.lindenma...@sap.com>>; core-libs- > > >>>>>> d...@openjdk.java.net <mailto: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 > > <mailto:core-libs-dev@openjdk.java.net> > > >>>>>>> Cc: Baesken, Matthias <matthias.baes...@sap.com > > <mailto: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. > >