RFR[9](XXS): 8037013: [TESTBUG] Fix test/java/lang/ClassLoader/Assert.sh on AIX
Volker Simonis
volker.simonis at gmail.com
Mon Mar 17 10:48:08 UTC 2014
Hi Mandy,
sorry for the delay. You are right, the shell script isn't needed. I
updated the change as suggested by you in the mail you referenced.
Here's the new webrev (tested on Linux/AIX/Solaris):
http://cr.openjdk.java.net/~simonis/webrevs/8037013_2/webrev/
OK, to push now?
Thank you and best regards,
Volker
On Tue, Mar 11, 2014 at 6:56 PM, Volker Simonis
<volker.simonis at gmail.com> wrote:
> Hi Mandy,
>
> thanks for the review.
>
> I'll look into the possibility of removing Assert.sh altogether and
> post a new webrev soon.
>
> Regards,
> Volker
>
>
> On Tue, Mar 11, 2014 at 2:59 AM, Mandy Chung <mandy.chung at oracle.com> wrote:
>>
>> On 3/10/2014 11:55 AM, Mandy Chung wrote:
>>>
>>> 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 am able to access the webrev now - looks good.
>>
>>
>>> 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?
>>>
>>
>> It'd be even better if we could get rid of Assert.sh.
>>
>> thanks
>> Mandy
>>
>>> Mandy
>>> [1]
>>> http://mail.openjdk.java.net/pipermail/core-libs-dev/2014-February/024995.html
>>
>>
More information about the core-libs-dev
mailing list