[prev in list] [next in list] [prev in thread] [next in thread]
List: openjdk-2d-dev
Subject: [OpenJDK 2D-Dev] [PATCH] Request for review: 6888734, regression test fails when java.security.manag
From: Igor.Nekrestyanov () Sun ! COM (Igor Nekrestyanov)
Date: 2009-12-08 18:53:07
Message-ID: 4B1EA093.1040402 () sun ! com
[Download RAW message or body]
As Dmitri mentioned you need to move policy file too. Otherwise test
will not work in new location when run with jtreg.
Rest looks good to me.
-igor
On 12/7/09 12:12 PM, Roman Kennke wrote:
> Hi Phil,
>
> Am Montag, den 30.11.2009, 13:37 -0800 schrieb Phil Race:
>
>> Roman Kennke wrote:
>>
>>> I added the (previously closed) testcase, keeping it in the same
>>> relative directory location:
>>>
>>> http://cr.openjdk.java.net/~rkennke/6888734/webrev.01/
>>>
>>> Is this ok?
>>>
>>>
>> Almost. Reg tests don't have the classpath exception.
>>
> Fixed.
>
>
>> And probably the copyright year should be a range, starting
>> with whatever the existing year is.
>>
> This file is added to the closed repository in 2007, with the initial
> commit, it doesn't say anything how old this file is. I put it in
> OpenJDK in 2009, this is why I say so in the header. What else can I do?
>
> Also, we need one more review, Igor, can you have a look?
>
> http://cr.openjdk.java.net/~rkennke/6888734/webrev.02/
>
> Thanks, Roman
>
>
>
>
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic