RFR(S/L): 8028537: PPC64: Updated the JDK regression tests to run on AIX
Alan Bateman
Alan.Bateman at oracle.com
Fri Jan 17 09:48:46 UTC 2014
On 15/01/2014 16:42, Volker Simonis wrote:
> Hi Alan,
>
> thanks for the suggestion. That's fine for me. I've copied the empty
> SCTP stubs from the macosx to the aix directory as well and updated
> the make file accordingly (in the patch for "8031581: PPC64: Addons
> and fixes for AIX to pass the jdk regression tests").
>
> Therefore, the changes to the three tests:
>
> test/com/sun/nio/sctp/SctpChannel/Util.java
> test/com/sun/nio/sctp/SctpMultiChannel/Util.java
> test/com/sun/nio/sctp/SctpServerChannel/Util.java
>
> can be considered obsolete.
Thanks, I think this makes the most sense.
I looked through the rest of this webrev and the update to the tests
looks fine.
One general comment is that for many of these shell tests (that survive
the current effort to replace them) is that we could move the Unix
handling into the match any case so that we don't have to list each of
Linux, SunOS, Darwin, ... I think this came up when the OS X port was
brought in but there wasn't any follow-up on it. I am not suggesting you
do this here, it's just a comment as I see same change to so many tests.
A minor comment on SBC.java is that it could just catch
UnsupportedOperationException on L238, that would avoid needing to check
os.name.
A really minor comment on the updates to ProblemList.txt is that the JMX
test should probably be in the jdk_jmx section (it's just a convention
that we've been using, it doesn't of course really matter where tests
are listed).
-Alan
More information about the core-libs-dev
mailing list