RFR(XXS): 8037013: [TESTBUG] Fix test/java/lang/ClassLoader/Assert.sh on AIX
mandy.chung at oracle.com
Tue Mar 11 01:59:35 UTC 2014
On 3/10/2014 11:55 AM, Mandy Chung wrote:
> Hi Volker,
> On 3/10/14 11:16 AM, Volker Simonis wrote:
>> could you please review the following trivial change which adds AIX to
>> the list of known OSs in test/java/lang/ClassLoader/Assert.sh:
> 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 am able to access the webrev now - looks good.
> I raised in the code review of 7152892  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?
It'd be even better if we could get rid of Assert.sh.
More information about the core-libs-dev