hg: jdk8/tl/jdk: 8028505: Put sun/jvmstat/monitor/MonitoredVm/MonitorVmStartTerminate.sh on ProblemList.txt

2013-11-19 Thread erik . gahlin
Changeset: d6195774dd1f Author:egahlin Date: 2013-11-19 11:47 +0100 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/d6195774dd1f 8028505: Put sun/jvmstat/monitor/MonitoredVm/MonitorVmStartTerminate.sh on ProblemList.txt Reviewed-by: alanb ! test/ProblemList.txt

hg: jdk8/tl/jdk: 7086879: java/net/InetAddress/CheckJNI.java hangs on Linux when IPv6 enabled

2013-11-19 Thread chris . hegarty
Changeset: d5ddde25d107 Author:tyan Date: 2013-11-19 13:46 + URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/d5ddde25d107 7086879: java/net/InetAddress/CheckJNI.java hangs on Linux when IPv6 enabled Reviewed-by: chegar ! test/ProblemList.txt

hg: jdk8/tl/jdk: 8028478: Re-visit JPRT testsets to make it easier to run subsets of the tests

2013-11-19 Thread alan . bateman
Changeset: 2e574350a2b6 Author:alanb Date: 2013-11-19 14:08 + URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/2e574350a2b6 8028478: Re-visit JPRT testsets to make it easier to run subsets of the tests Reviewed-by: dholmes, sla, tbell ! test/Makefile

hg: jdk8/tl: 8028478: Re-visit JPRT testsets to make it easier to run subsets of the tests

2013-11-19 Thread alan . bateman
Changeset: 9937f406e27e Author:alanb Date: 2013-11-19 14:11 + URL: http://hg.openjdk.java.net/jdk8/tl/rev/9937f406e27e 8028478: Re-visit JPRT testsets to make it easier to run subsets of the tests Reviewed-by: dholmes, sla, tbell ! make/jprt.properties ! test/Makefile

[8] RFR 8028377: test/sun/security/provider/KeyStore/DKSTest.sh attempts to write to ${test.src}

2013-11-19 Thread Vincent Ryan
Please review the following test fix: Bug: https://bugs.openjdk.java.net/browse/JDK-8028377 Webrev: http://cr.openjdk.java.net/~vinnie/8028377/webrev.00/ It now writes temporary keystores in the test scratch area (not the test source area). Thanks.

Re: [8] RFR 8028377: test/sun/security/provider/KeyStore/DKSTest.sh attempts to write to ${test.src}

2013-11-19 Thread Alan Bateman
On 19/11/2013 14:16, Vincent Ryan wrote: Please review the following test fix: Bug: https://bugs.openjdk.java.net/browse/JDK-8028377 Webrev: http://cr.openjdk.java.net/~vinnie/8028377/webrev.00/ It now writes temporary keystores in the test scratch area (not the test source area). This looks

Re: [8] RFR 8028377: test/sun/security/provider/KeyStore/DKSTest.sh attempts to write to ${test.src}

2013-11-19 Thread Weijun Wang
Why not just put the file in the current directory? That's the real scratch dir. Or the cfg file needs an absolute path? Also, is it necessary to remove the files in finally? If the test fails, the files will be retained by jtreg so we can look into the reason. Or, these lines are added to

Re: [8] RFR 8028377: test/sun/security/provider/KeyStore/DKSTest.sh attempts to write to ${test.src}

2013-11-19 Thread Vincent Ryan
Thanks Alan. I've changed to using the two-arg constructor for File and I've updated the webrev in place. On 19/11/2013 14:23, Alan Bateman wrote: On 19/11/2013 14:16, Vincent Ryan wrote: Please review the following test fix: Bug: https://bugs.openjdk.java.net/browse/JDK-8028377 Webrev:

hg: jdk8/tl/jdk: 8028583: Add helper methods to test libraries

2013-11-19 Thread sean . coffey
Changeset: d1bb85f0a45a Author:coffeys Date: 2013-11-19 14:47 + URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/d1bb85f0a45a 8028583: Add helper methods to test libraries Reviewed-by: chegar ! test/java/rmi/testlibrary/TestLibrary.java !

hg: jdk8/tl/jdk: 8023978: [TEST_BUG] launcher tests must exclude platforms without server vm

2013-11-19 Thread kumar . x . srinivasan
Changeset: 40462a41b41b Author:ksrini Date: 2013-11-19 07:10 -0800 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/40462a41b41b 8023978: [TEST_BUG] launcher tests must exclude platforms without server vm Reviewed-by: dholmes, mchung ! test/tools/launcher/ExecutionEnvironment.java

Re: [8] RFR 8028377: test/sun/security/provider/KeyStore/DKSTest.sh attempts to write to ${test.src}

2013-11-19 Thread Vincent Ryan
On 19/11/2013 14:30, Weijun Wang wrote: Why not just put the file in the current directory? That's the real scratch dir. Or the cfg file needs an absolute path? Good point Max. The cfg file does require an absolute path so I've changed the test to now write to {user.dir}. And I've updated

hg: jdk8/tl/jdk: 8028094: TEST_BUG: java/lang/ProcessBuilder/Basic.java leaves sleep 6666 processes behind

2013-11-19 Thread chris . hegarty
Changeset: cfbee8ee71bf Author:bvaidya Date: 2013-11-19 15:31 + URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/cfbee8ee71bf 8028094: TEST_BUG: java/lang/ProcessBuilder/Basic.java leaves sleep processes behind Reviewed-by: chegar !

hg: jdk8/tl/jdk: 2 new changesets

2013-11-19 Thread vincent . x . ryan
Changeset: e8daf5a83e42 Author:vinnie Date: 2013-11-19 15:39 + URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/e8daf5a83e42 8028377: test/sun/security/provider/KeyStore/DKSTest.sh attempts to write to ${test.src} Reviewed-by: alanb, weijun !

hg: jdk8/tl/jdk: 8028581: [TESTBUG] java/net/Socket/LingerTest.java failing

2013-11-19 Thread michael . x . mcmahon
Changeset: 79e975dfeb8a Author:michaelm Date: 2013-11-19 17:49 + URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/79e975dfeb8a 8028581: [TESTBUG] java/net/Socket/LingerTest.java failing Reviewed-by: alanb ! test/java/net/Socket/LingerTest.java

hg: jdk8/tl/jdk: 8027900: pack200 option is broken due to the incorrect makefile definition for its driver

2013-11-19 Thread alexander . zuev
Changeset: 5aa853ca08a8 Author:kizune Date: 2013-11-19 22:05 +0400 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/5aa853ca08a8 8027900: pack200 option is broken due to the incorrect makefile definition for its driver Reviewed-by: ksrini, ihse ! make/CompileLaunchers.gmk

hg: jdk8/tl/jdk: 8015571: OCSP validation fails if ocsp.responderCertSubjectName is set

2013-11-19 Thread vincent . x . ryan
Changeset: f8b24e1a609e Author:vinnie Date: 2013-11-19 17:55 + URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/f8b24e1a609e 8015571: OCSP validation fails if ocsp.responderCertSubjectName is set Reviewed-by: mullan, xuelei !

hg: jdk8/tl/jdk: 8028565: Remove java/lang/management/ThreadMXBean/ThreadStateTest.java from ProblemList.txt

2013-11-19 Thread mandy . chung
Changeset: 48c61808374f Author:mchung Date: 2013-11-19 10:19 -0800 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/48c61808374f 8028565: Remove java/lang/management/ThreadMXBean/ThreadStateTest.java from ProblemList.txt Reviewed-by: sla ! test/ProblemList.txt

hg: jdk8/tl/jdk: 8028141: test/sun/management/jmxremote/bootstrap/LocalManagementTest|CustomLauncherTest.java failing again

2013-11-19 Thread roger . riggs
Changeset: 3f47e393e1dd Author:rriggs Date: 2013-11-19 13:20 -0500 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/3f47e393e1dd 8028141: test/sun/management/jmxremote/bootstrap/LocalManagementTest|CustomLauncherTest.java failing again Summary: Correct to use the test.class.path

hg: jdk8/tl/jdk: 8028185: XMLFormatter.format emits incorrect year

2013-11-19 Thread daniel . fuchs
Changeset: 67d742c75971 Author:dfuchs Date: 2013-11-19 20:10 +0100 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/67d742c75971 8028185: XMLFormatter.format emits incorrect year Summary: Fixes a regression where the year in the date was increased by 1900. Reviewed-by: alanb, mchung

hg: jdk8/tl/jdk: 8005202: java/util/logging/CheckLockLocationTest.java fail on solars_10

2013-11-19 Thread daniel . fuchs
Changeset: 059530c5ae9a Author:dfuchs Date: 2013-11-19 22:28 +0100 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/059530c5ae9a 8005202: java/util/logging/CheckLockLocationTest.java fail on solars_10 Summary: this test has been seen failing on Solaris 10, presumably because it was

hg: jdk8/tl/jdk: 8028631: Improve the test coverage to the pathname handling on unix-like platforms

2013-11-19 Thread dan . xu
Changeset: f496565c4eec Author:dxu Date: 2013-11-19 13:22 -0800 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/f496565c4eec 8028631: Improve the test coverage to the pathname handling on unix-like platforms Summary: Add GeneralSolaris.java testcase and fix the concurrency issue

Re: [8] RFR 8028377: test/sun/security/provider/KeyStore/DKSTest.sh attempts to write to ${test.src}

2013-11-19 Thread Valerie (Yu-Ching) Peng
If the test passed, these temporary files should be removed, shouldn't they? Thanks, Valerie On 11/19/13 07:29, Vincent Ryan wrote: On 19/11/2013 14:30, Weijun Wang wrote: Why not just put the file in the current directory? That's the real scratch dir. Or the cfg file needs an absolute path?

Re: [8] RFR 8028377: test/sun/security/provider/KeyStore/DKSTest.sh attempts to write to ${test.src}

2013-11-19 Thread Vincent Ryan
Hello Valerie, They get written to the scratch directory for the testrun so I don't think it is necessary to delete them explicitly. I'll confirm that. Thanks. On 19/11/2013 21:58, Valerie (Yu-Ching) Peng wrote: If the test passed, these temporary files should be removed, shouldn't they?

hg: jdk8/tl/jdk: 8028638: java/rmi/activation/Activatable/checkRegisterInLog/CheckRegisterInLog.java fails

2013-11-19 Thread stuart . marks
Changeset: 19d2e9649138 Author:smarks Date: 2013-11-19 15:05 -0800 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/19d2e9649138 8028638: java/rmi/activation/Activatable/checkRegisterInLog/CheckRegisterInLog.java fails Reviewed-by: lancea ! test/java/rmi/testlibrary/RMID.java

hg: jdk8/tl/langtools: 8028504: javac generates LocalVariableTable even with -g:none

2013-11-19 Thread vicente . romero
Changeset: 66bcd5d4b3d1 Author:vromero Date: 2013-11-19 23:35 + URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/66bcd5d4b3d1 8028504: javac generates LocalVariableTable even with -g:none Reviewed-by: jjg, jlahoda !

Thoughts on possible options to JDK-8027598

2013-11-19 Thread Rajan Halade
I am working towards fixing JDK-8027598 https://bugs.openjdk.java.net/browse/JDK-8027598, there are multiple options available here and would appreciate your thoughts on this. It was filed to address the issue at large reported inJDK-8027526 https://bugs.openjdk.java.net/browse/JDK-8027526.

Re: Thoughts on possible options to JDK-8027598

2013-11-19 Thread Xuelei Fan
On 11/20/2013 8:19 AM, Rajan Halade wrote: I am working towards fixing JDK-8027598 https://bugs.openjdk.java.net/browse/JDK-8027598, there are multiple options available here and would appreciate your thoughts on this. It was filed to address the issue at large reported inJDK-8027526

Code Review Request for 7200306: SunPKCS11 provider delays the check of DSA key size for SHA1withDSA to sign() instead of init()

2013-11-19 Thread Valerie (Yu-Ching) Peng
Can someone please help review my fixes for 7200306: SunPKCS11 provider delays the check of DSA key size for SHA1withDSA to sign() instead of init()? Native PKCS11 libraries don't seem to check the key during the initialization calls (triggered by initSign()/initVerify()). Rather, it errors

Re: Thoughts on possible options to JDK-8027598

2013-11-19 Thread Bradford Wetmore
On 11/19/2013 4:19 PM, Rajan Halade wrote: I am working towards fixing JDK-8027598 https://bugs.openjdk.java.net/browse/JDK-8027598, there are multiple options available here and would appreciate your thoughts on this. It was filed to address the issue at large reported inJDK-8027526

Re: Thoughts on possible options to JDK-8027598

2013-11-19 Thread Bradford Wetmore
On 11/19/2013 5:07 PM, Xuelei Fan wrote: On 11/20/2013 8:19 AM, Rajan Halade wrote: I am working towards fixing JDK-8027598 https://bugs.openjdk.java.net/browse/JDK-8027598, there are multiple options available here and would appreciate your thoughts on this. It was filed to address the issue