hg: jdk8/tl/jdk: 7184815: [macosx] Need to read Kerberos config in files

2012-08-28 Thread weijun . wang
Changeset: c4c69b4d9ace Author:weijun Date: 2012-08-29 11:03 +0800 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/c4c69b4d9ace 7184815: [macosx] Need to read Kerberos config in files Reviewed-by: valeriep ! src/share/classes/sun/security/krb5/Config.java

7u8 Code Review Request for 7107613, 7107616, 7185471

2012-08-28 Thread Seán Coffey
Looking for a code review around the following perf. related backports to JDK 7u8. The changesets didn't apply cleanly but there was no major code differences encountered while porting. Builds and security tests ran fine. 7107616: scalability blocker in javax.crypto.JceSecurityManager jdk 8 ch

hg: jdk8/tl/jdk: 7194608: add VerifyLocalVariableTableOnRetransformTest.sh to Problem.list

2012-08-28 Thread daniel . daugherty
Changeset: bfd5ecb1b4aa Author:dcubed Date: 2012-08-28 09:40 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/bfd5ecb1b4aa 7194608: add VerifyLocalVariableTableOnRetransformTest.sh to Problem.list Reviewed-by: alanb ! test/ProblemList.txt

Re: JDK 8 Code review request for 7179715: Reason of CertPathValidatorException should be UNDETERMINED_REVOCATION_STATUS if OCSP request failed

2012-08-28 Thread Vincent Ryan
Webrev looks fine. On 08/28/12 01:20 PM, Sean Mullan wrote: Vinnie, Weijun or Xuelei, Could one of you take a quick look at this and let me know if it looks ok? Thanks, Sean On 8/27/12 4:23 PM, Sean Mullan wrote: Please review my webrev for 7179715: http://cr.openjdk.java.net/~mullan/webrev

hg: jdk8/tl/jdk: 2 new changesets

2012-08-28 Thread sean . mullan
Changeset: 8b90182f2b33 Author:mullan Date: 2012-08-28 08:43 -0400 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/8b90182f2b33 7192896: Reason of CertPathValidatorException should be UNDETERMINED_REVOCATION_STATUS if OCSP request failed Reviewed-by: xuelei ! src/share/classes/sun

Re: JDK 8 Code review request for 7179715: Reason of CertPathValidatorException should be UNDETERMINED_REVOCATION_STATUS if OCSP request failed

2012-08-28 Thread Sean Mullan
On 8/28/12 8:20 AM, Xuelei Fan wrote: > On 8/28/2012 4:23 AM, Sean Mullan wrote: >> Please review my webrev for 7179715: >> > Is it 7192896? Oops, yes, cut-and-paste error. > >> http://cr.openjdk.java.net/~mullan/webrevs/7192896/webrev.00/ >> > > Looks fine to me. Thanks. --Sean > > Xuelei

Re: JDK 8 Code review request for 7179715: Reason of CertPathValidatorException should be UNDETERMINED_REVOCATION_STATUS if OCSP request failed

2012-08-28 Thread Xuelei Fan
On 8/28/2012 4:23 AM, Sean Mullan wrote: > Please review my webrev for 7179715: > Is it 7192896? > http://cr.openjdk.java.net/~mullan/webrevs/7192896/webrev.00/ > Looks fine to me. Xuelei > The bugid is not accessible for some reason. Essentially this is a simple fix > to > set the reason of

Re: JDK 8 Code review request for 7179715: Reason of CertPathValidatorException should be UNDETERMINED_REVOCATION_STATUS if OCSP request failed

2012-08-28 Thread Sean Mullan
Vinnie, Weijun or Xuelei, Could one of you take a quick look at this and let me know if it looks ok? Thanks, Sean On 8/27/12 4:23 PM, Sean Mullan wrote: > Please review my webrev for 7179715: > > http://cr.openjdk.java.net/~mullan/webrevs/7192896/webrev.00/ > > The bugid is not accessible for

hg: jdk8/tl/jdk: 6962637: TEST_BUG: java/io/File/MaxPathLength.java may fail in busy system

2012-08-28 Thread alan . bateman
Changeset: c5099c988cce Author:alanb Date: 2012-08-28 11:12 +0100 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/c5099c988cce 6962637: TEST_BUG: java/io/File/MaxPathLength.java may fail in busy system Reviewed-by: dholmes, alanb Contributed-by: Eric Wang ! test/ProblemList.txt !

hg: jdk8/tl/jdk: 7194035: update tests for upcoming changes for jtreg

2012-08-28 Thread jonathan . gibbons
Changeset: 997e0d6238b7 Author:jjg Date: 2012-08-28 10:31 +0100 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/997e0d6238b7 7194035: update tests for upcoming changes for jtreg Reviewed-by: alanb, sspitsyn ! test/sun/tools/common/ApplicationSetup.sh ! test/sun/tools/jps/jps-Vvml_2

hg: jdk8/tl/jdk: 7194032: update tests for upcoming changes for jtreg

2012-08-28 Thread jonathan . gibbons
Changeset: 06d0478023ca Author:jjg Date: 2012-08-28 10:29 +0100 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/06d0478023ca 7194032: update tests for upcoming changes for jtreg Reviewed-by: alanb, iris ! test/java/rmi/activation/Activatable/extLoadedImpl/ext.sh ! test/java/rmi/reg

hg: jdk8/tl/jdk: 7194472: FileKeyTab.java test fails on Windows

2012-08-28 Thread weijun . wang
Changeset: fe496675b5e7 Author:weijun Date: 2012-08-28 17:25 +0800 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/fe496675b5e7 7194472: FileKeyTab.java test fails on Windows Reviewed-by: alanb ! test/sun/security/krb5/auto/FileKeyTab.java

Re: Code review request: 7194472: FileKeyTab.java test fails on Windows

2012-08-28 Thread Alan Bateman
On 28/08/2012 06:32, Weijun Wang wrote: Sorry, there is a problem in the test of my recent fix at http://hg.openjdk.java.net/jdk8/tl/jdk/rev/61ddc8ce7f3b Here is a webrev http://cr.openjdk.java.net/~weijun/7194472/webrev.00/ "/" is treated as a universal File.separatorChar in JAAS config