RFR(S): 8194869: [TESTBUG][aix, s390] Adapt tests to platforms.
David Holmes
david.holmes at oracle.com
Thu Jan 18 01:17:41 UTC 2018
Hi Volker,
On 18/01/2018 9:21 AM, Volker Simonis wrote:
>
> David Holmes <david.holmes at oracle.com <mailto: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.
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/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
> <mailto:david.holmes at oracle.com>]
> >> Sent: Mittwoch, 17. Januar 2018 12:55
> >> To: Lindenmaier, Goetz <goetz.lindenmaier at sap.com
> <mailto:goetz.lindenmaier at sap.com>>; Volker Simonis
> >> <volker.simonis at gmail.com <mailto:volker.simonis at gmail.com>>
> >> Cc: core-libs-dev at openjdk.java.net
> <mailto:core-libs-dev at openjdk.java.net>; Baesken, Matthias
> >> <matthias.baesken at sap.com <mailto:matthias.baesken at 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.simonis at gmail.com
> <mailto:volker.simonis at gmail.com>]
> >>>> Sent: Dienstag, 16. Januar 2018 19:28
> >>>> To: Lindenmaier, Goetz <goetz.lindenmaier at sap.com
> <mailto:goetz.lindenmaier at sap.com>>
> >>>> Cc: Baesken, Matthias <matthias.baesken at sap.com
> <mailto:matthias.baesken at sap.com>>; core-libs-
> >>>> dev at openjdk.java.net <mailto: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 <mailto: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
> <mailto:goetz.lindenmaier at sap.com>>; core-libs-
> >>>>>> dev at openjdk.java.net <mailto: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
> <mailto:core-libs-dev at openjdk.java.net>
> >>>>>>> Cc: Baesken, Matthias <matthias.baesken at sap.com
> <mailto: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