<Langtools> [7u] Review Request: JDK-8064454 Fix type in langtools for tools/javac/innerClassFile/Driver.sh

Maurizio Cimadamore maurizio.cimadamore at oracle.com
Thu Jan 22 13:35:17 UTC 2015


It seems like the problematic rm in the last line has been removed i.e. 
in JDK 8?

http://hg.openjdk.java.net/jdk8u/jdk8u-dev/langtools/file/2d2b2be57c78/test/tools/javac/innerClassFile/Driver.sh

Perhaps, instead of tweaking permission, we should just do things the 
JDK 8 way?

Maurizio

On 12/01/15 07:04, pooja chopra wrote:
> Hi All,
> Please review below fix .
>
> 8064454 [TEST_BUG] Test tools/javac/innerClassFile/Driver.sh fails 
> with exit code 1
> Test bug fix.
> https://bugs.openjdk.java.net/browse/JDK-8064454
> The webrev is: http://cr.openjdk.java.net/~kshefov/8064454/webrev.00/
>
> Regards,
> Pooja
> On 12/5/2014 11:09 AM, pooja chopra wrote:
>> Hi All,
>> Gentle Reminder . Please review below fix .
>>
>> 8064454 [TEST_BUG] Test tools/javac/innerClassFile/Driver.sh fails 
>> with exit code 1
>> Test bug fix.
>> https://bugs.openjdk.java.net/browse/JDK-8064454
>> The webrev is: http://cr.openjdk.java.net/~kshefov/8064454/webrev.00/
>>
>> Regards,
>> Pooja
>> On 11/10/2014 10:29 PM, pooja chopra wrote:
>>>
>>> Hello,
>>> Please review a fix for the issue:
>>> 8064454 [TEST_BUG] Test tools/javac/innerClassFile/Driver.sh fails 
>>> with exit code 1
>>> Test bug fix.
>>> https://bugs.openjdk.java.net/browse/JDK-8064454
>>> The webrev is: http://cr.openjdk.java.net/~kshefov/8064454/webrev.00/
>>>
>>> Thanks
>>> Pooja
>>
>



More information about the jdk7u-dev mailing list