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?


More information about the core-libs-dev mailing list