Re: [OpenJDK 2D-Dev] [PATCH] Request for review: 6888734, regression test fails when java.security.manager is enabled

2010-02-01 Thread Dmitri Trembovetski
Roman Kennke wrote: Looks ok to me except the copyright on the test should probably be 2010 now =) Arg! I knew it! http://cr.openjdk.java.net/~rkennke/6888734/webrev.04/ Now don't tell me I have to put 'Oracle' in there! ;-) Ship it before we had to go through that exercise.. =) Dmit

Re: [OpenJDK 2D-Dev] [PATCH] Request for review: 6888734, regression test fails when java.security.manager is enabled

2010-02-01 Thread Igor Nekrestyanov
ok with me. -igor On 2/1/10 12:16 PM, Roman Kennke wrote: Looks ok to me except the copyright on the test should probably be 2010 now =) Arg! I knew it! http://cr.openjdk.java.net/~rkennke/6888734/webrev.04/ Now don't tell me I have to put 'Oracle' in there! ;-) /Roman Dmitr

Re: [OpenJDK 2D-Dev] [PATCH] Request for review: 6888734, regression test fails when java.security.manager is enabled

2010-02-01 Thread Roman Kennke
> Looks ok to me except the copyright on the test should probably be 2010 now =) Arg! I knew it! http://cr.openjdk.java.net/~rkennke/6888734/webrev.04/ Now don't tell me I have to put 'Oracle' in there! ;-) /Roman > >Dmitri > > Roman Kennke wrote: > > Am Dienstag, den 08.12.2009, 10:53 -

Re: [OpenJDK 2D-Dev] [PATCH] Request for review: 6888734, regression test fails when java.security.manager is enabled

2010-02-01 Thread Dmitri Trembovetski
Looks ok to me except the copyright on the test should probably be 2010 now =) Dmitri Roman Kennke wrote: Am Dienstag, den 08.12.2009, 10:53 -0800 schrieb Igor Nekrestyanov: As Dmitri mentioned you need to move policy file too. Otherwise test will not work in new location when run with jt

Re: [OpenJDK 2D-Dev] [PATCH] Request for review: 6888734, regression test fails when java.security.manager is enabled

2010-02-01 Thread Roman Kennke
Am Dienstag, den 08.12.2009, 10:53 -0800 schrieb Igor Nekrestyanov: > As Dmitri mentioned you need to move policy file too. Otherwise test > will not work in new location when run with jtreg. So I finally found some time to get back at this. I added the policy file too: http://cr.openjdk.java.n