RFR[9](XXS): 8037013: [TESTBUG] Fix test/java/lang/ClassLoader/Assert.sh on AIX

Mandy Chung mandy.chung at oracle.com
Mon Mar 10 18:55:26 UTC 2014


Hi Volker,

On 3/10/14 11:16 AM, Volker Simonis wrote:
> Hi,
>
> could you please review the following trivial change which adds AIX to
> the list of known OSs in test/java/lang/ClassLoader/Assert.sh:
>
> https://bugs.openjdk.java.net/browse/JDK-8037013
> http://cr.openjdk.java.net/~simonis/webrevs/8037013/
>

The link to your webrev is not accessible and could possibly due to the 
server issue.  I guess your fix is to add AIX in the Assert.sh test.  
Can you paste the diff in mail?  I can review it.

I raised in the code review of 7152892 [1] that I believe this test 
doesn't need to be a shell test.  I don't know if Rob has started 
looking into it (he missed my review comment when he pushed the fix).   
I wonder if you would like to take the opportunity to investigate if 
Assert.sh can be removed?

Mandy
[1] 
http://mail.openjdk.java.net/pipermail/core-libs-dev/2014-February/024995.html 



More information about the ppc-aix-port-dev mailing list