RFR(S): 8194869: [TESTBUG][aix, s390] Adapt tests to platforms. (original) (raw)

Volker Simonis volker.simonis at gmail.com
Wed Jan 17 23:21:28 UTC 2018

David Holmes <david.holmes at 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.

Regards, Volker

[1] https://www.ibm.com/support/knowledgecenter/en/ssw_i5_54/rzalf/rzalfintro.htm

Thanks, David > I'll post it on awt-dev, too. > > Best regards, > Goetz. > >> -----Original Message----- >> From: David Holmes [mailto:david.holmes at oracle.com] >> Sent: Mittwoch, 17. Januar 2018 12:55 >> To: Lindenmaier, Goetz <goetz.lindenmaier at sap.com>; Volker Simonis >> <volker.simonis at gmail.com> >> Cc: core-libs-dev at openjdk.java.net; Baesken, Matthias >> <matthias.baesken at sap.com> >> Subject: Re: RFR(S): 8194869: [TESTBUG][aix, s390] Adapt tests to platforms. >> >> Hi Goetz, >> >> test/failurehandler/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.simonis at gmail.com] >>>> Sent: Dienstag, 16. Januar 2018 19:28 >>>> To: Lindenmaier, Goetz <goetz.lindenmaier at sap.com> >>>> Cc: Baesken, Matthias <matthias.baesken at sap.com>; core-libs- >>>> dev at 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.lindenmaier at 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.lindenmaier at sap.com>; core-libs- >>>>>> dev at 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 at openjdk.java.net >>>>>>> Cc: Baesken, Matthias <matthias.baesken at 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.

More information about the core-libs-dev mailing list