Re: [7u] review and approval request for 8007515: update RMI tests on ProblemList.txt

2013-02-12 Thread Stuart Marks
Change pushed. We now have 134 RMI tests passing in jdk7u, with only 4 excluded -- same as jdk8! Regarding the mailing list, we should probably replace jdk-regt...@sun.com with an OpenJDK mailing list. The problem is, which one? Core-libs-dev, jdk8-dev, jdk7u-dev, discuss, and quality-discuss

hg: jdk7u/jdk7u-dev/jdk: 8007515: TEST_BUG: update ProblemList.txt and TEST.ROOT in jdk7u-dev to match jdk8

2013-02-12 Thread stuart . marks
Changeset: a724ea5862a7 Author:smarks Date: 2013-02-12 14:41 -0800 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/a724ea5862a7 8007515: TEST_BUG: update ProblemList.txt and TEST.ROOT in jdk7u-dev to match jdk8 Reviewed-by: alanb, mchung, dmocek ! test/ProblemList.txt ! te

Re: FAILS: test/sun/management/jdp/JdpTest.sh: test: argument expected

2013-02-12 Thread Dmitry Samersoff
Mandy, On 2013-02-13 02:25, Mandy Chung wrote: > You didn't include jdk_management test target in your jprt job and your > test was not run at all. jdk_management is not included in the default > due to its test reliability issue. Thank you for letting me know! > Sounds like you didn't run the

Re: FAILS: test/sun/management/jdp/JdpTest.sh: test: argument expected

2013-02-12 Thread alejandro murillo
Dmitry, you have to use "-testset core" to run this test via JPRT. It's not run on regular jobs. Alejandro On 2/12/2013 3:18 PM, Dmitry Samersoff wrote: Kelly, Do you have an idea why I didn't see this failure under jprt? http://sthjprt.se.oracle.com/archives/2013/02/2013-02-08-192233.dsa

Re: FAILS: test/sun/management/jdp/JdpTest.sh: test: argument expected

2013-02-12 Thread Mandy Chung
You didn't include jdk_management test target in your jprt job and your test was not run at all. jdk_management is not included in the default due to its test reliability issue. Sounds like you didn't run the new shell tests via jtreg on your local machine to make sure it passes before you sub

Re: FAILS: test/sun/management/jdp/JdpTest.sh: test: argument expected

2013-02-12 Thread Dmitry Samersoff
Chris, I'm not able to reproduce it locally. Do you have a link to jprt job you see this failure? -Dmitry On 2013-02-12 21:38, Chris Hegarty wrote: > Dmitry, > > This test is now failing on several platforms, on jdk8 and 7u-dev > > --- > > result: Passed. Compilation successful > > #sect

Re: FAILS: test/sun/management/jdp/JdpTest.sh: test: argument expected

2013-02-12 Thread Dmitry Samersoff
Kelly, Do you have an idea why I didn't see this failure under jprt? http://sthjprt.se.oracle.com/archives/2013/02/2013-02-08-192233.dsamerso.tl/ -Dmitry On 2013-02-12 21:52, Kelly O'Hair wrote: > +if [ ! -f ${_testclasses} ] > needs to be > +if [ ! -d ${_testclasses} ] > > > -kto >

Re: FAILS: test/sun/management/jdp/JdpTest.sh: test: argument expected

2013-02-12 Thread Dmitry Samersoff
Kelly, Good catch but this part of test is not executed under JTReg, and this mistake cause extra mkdir call but not a test error. -Dmitry On 2013-02-12 21:52, Kelly O'Hair wrote: > +if [ ! -f ${_testclasses} ] > needs to be > +if [ ! -d ${_testclasses} ] > > > -kto > > On Feb 12, 201

Re: [7u] review and approval request for 8007515: update RMI tests on ProblemList.txt

2013-02-12 Thread Darryl Mocek
Seems to me we need an Oracle equivalent. Is there already one for jtreg or JDK tests in general? Not a huge priority, but we should get rid of the sun.com email addresses when an easy opportunity arises. Darryl On 02/11/2013 06:06 PM, Stuart Marks wrote: Arrgh, ya caught me! :-) I was aware

Re: [7u] review and approval request for 8007515: update RMI tests on ProblemList.txt

2013-02-12 Thread Mandy Chung
On 2/11/13 3:02 PM, Stuart Marks wrote: http://cr.openjdk.java.net/~smarks/reviews/8007515/webrev.1/ Looks okay to me. Mandy

Re: FAILS: test/sun/management/jdp/JdpTest.sh: test: argument expected

2013-02-12 Thread Kelly O'Hair
+if [ ! -f ${_testclasses} ] needs to be +if [ ! -d ${_testclasses} ] -kto On Feb 12, 2013, at 9:38 AM, Chris Hegarty wrote: > Dmitry, > > This test is now failing on several platforms, on jdk8 and 7u-dev > > --- > > result: Passed. Compilation successful > > #section:shell > --

FAILS: test/sun/management/jdp/JdpTest.sh: test: argument expected

2013-02-12 Thread Chris Hegarty
Dmitry, This test is now failing on several platforms, on jdk8 and 7u-dev --- result: Passed. Compilation successful #section:shell --messages:(3/154)-- command: shell JdpTest.sh [--jtreg, --no-compile] reason: User specified action: run shell JdpTest.sh --jtreg --no-compil

hg: jdk7u/jdk7u-dev/jdk: 8007761: NTLM coding errors

2013-02-12 Thread weijun . wang
Changeset: cce9ada9668b Author:weijun Date: 2013-02-08 08:56 +0800 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/cce9ada9668b 8007761: NTLM coding errors Reviewed-by: chegar ! src/share/classes/com/sun/security/ntlm/Client.java ! src/share/classes/com/sun/security/ntlm/NT

hg: jdk7u/jdk7u-dev/jdk: 11 new changesets

2013-02-12 Thread chris . hegarty
Changeset: d0d5986f25ef Author:chegar Date: 2011-08-09 16:59 +0100 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/d0d5986f25ef 7076756: TEST_BUG: com/sun/jdi/BreakpointWithFullGC.sh fails to cleanup in Cygwin Reviewed-by: alanb, dcubed ! test/com/sun/jdi/ShellScaffold.sh

hg: jdk7u/jdk7u-dev/jdk: 8007786: JDK-8002048 testcase doesn't work on Solaris

2013-02-12 Thread dmitry . samersoff
Changeset: 4697f4725cee Author:dsamersoff Date: 2013-02-12 16:41 +0400 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/4697f4725cee 8007786: JDK-8002048 testcase doesn't work on Solaris Summary: test built in into Solaris shell doesn't have -e operator Reviewed-by: sla, sspi

Re: [7u] Request for approval for 8007786 - JDK-8002048 testcase doesn't work on Solaris

2013-02-12 Thread Seán Coffey
Approved. regards, Sean. On 12/02/2013 12:12, Dmitry Samersoff wrote: Requesting approval to push the following change to jdk7u: Summary Built in test function of Solaris /bin/sh doesn't have -e operator and doesn't allow a variable and a function with the same name. webrev: http://cr.openj

Re: Request for Bulk Approval - backport test stabilization change from JDK8

2013-02-12 Thread Chris Hegarty
Thank you Sean. -Chris. On 02/12/2013 12:24 PM, Seán Coffey wrote: Looks good Chris! Approved for jdk7u-dev. regards, Sean. On 12/02/2013 11:46, Chris Hegarty wrote: This is a bulk request for approval to backport some test stabilization change from JDK8 to 7u-dev. There are many more, this

Re: Request for Bulk Approval - backport test stabilization change from JDK8

2013-02-12 Thread Seán Coffey
Looks good Chris! Approved for jdk7u-dev. regards, Sean. On 12/02/2013 11:46, Chris Hegarty wrote: This is a bulk request for approval to backport some test stabilization change from JDK8 to 7u-dev. There are many more, this is just a start! All changes are in JDK8, and all changesets are i

[7u] Request for approval for 8007786 - JDK-8002048 testcase doesn't work on Solaris

2013-02-12 Thread Dmitry Samersoff
Requesting approval to push the following change to jdk7u: Summary Built in test function of Solaris /bin/sh doesn't have -e operator and doesn't allow a variable and a function with the same name. webrev: http://cr.openjdk.java.net/~dsamersoff/8007786.JDP-TEST2/webrev.01/jdk/ bug: http://bug

hg: jdk7u/jdk7u-dev/jdk: 8005932: Java 7 on mac os x only provides text clipboard formats

2013-02-12 Thread alexander . potochkin
Changeset: 6707e6b768ee Author:mcherkas Date: 2013-02-12 16:11 +0400 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/6707e6b768ee 8005932: Java 7 on mac os x only provides text clipboard formats Reviewed-by: alexp, denis ! src/macosx/lib/flavormap.properties + test/java/aw

Request for Bulk Approval - backport test stabilization change from JDK8

2013-02-12 Thread Chris Hegarty
This is a bulk request for approval to backport some test stabilization change from JDK8 to 7u-dev. There are many more, this is just a start! All changes are in JDK8, and all changesets are identical. 7076756: TEST_BUG: com/sun/jdi/BreakpointWithFullGC.sh fails to cleanup in Cygwin http://hg

Re: [7u-next] Request for approval for CR 7076756 - TEST_BUG: com/sun/jdi/BreakpointWithFullGC.sh fails to cleanup in Cygwin

2013-02-12 Thread Seán Coffey
Approved, regards, Sean. On 12/02/2013 10:26, Chris Hegarty wrote: This is a request for approval to backport a test stabilization change from JDK8 to 7u-dev. http://bugs.sun.com/view_bug.do?bug_id=7076756 JDK8 changeset: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/cf203f293b4e JDK8 review

hg: jdk7u/jdk7u-dev/jdk: 6 new changesets

2013-02-12 Thread chris . hegarty
Changeset: 10eae5966c2d Author:chegar Date: 2012-09-07 14:00 +0100 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/10eae5966c2d 7032247: java/net/InetAddress/GetLocalHostWithSM.java fails if hostname resolves to loopback address Summary: TESTBUG Reviewed-by: chegar, alanb C

[7u-next] Request for approval for CR 7076756 - TEST_BUG: com/sun/jdi/BreakpointWithFullGC.sh fails to cleanup in Cygwin

2013-02-12 Thread Chris Hegarty
This is a request for approval to backport a test stabilization change from JDK8 to 7u-dev. http://bugs.sun.com/view_bug.do?bug_id=7076756 JDK8 changeset: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/cf203f293b4e JDK8 review thread: http://mail.openjdk.java.net/pipermail/serviceability-dev/20

Re: [7u-next] Request for approval for CR 7073295 - TEST_BUG: test/java/lang/instrument/ManifestTest.sh causing havoc (win)

2013-02-12 Thread Seán Coffey
Nice work on the test stabilisation front Chris! Approved, regards, Sean. On 12/02/2013 10:13, Chris Hegarty wrote: This is a request for approval to backport a test stabilization change from JDK8 to 7u-dev. http://bugs.sun.com/view_bug.do?bug_id=7073295 JDK8 changeset: http://hg.openjdk.

Re: [7u-next] Request for approval for CR 7044870 - java/nio/channels/DatagramChannel/SelectWhenRefused.java failed on SUSE Linux 10

2013-02-12 Thread Seán Coffey
Approved, regards, Sean. On 12/02/2013 10:06, Chris Hegarty wrote: This is a request for approval to backport a test stabilization change from JDK8 to 7u-dev. http://bugs.sun.com/view_bug.do?bug_id=7044870 JDK8 changeset: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/c9e78769c0da JDK8 review

Re: [7u-next] Request for approval for CR 6776144 - java/lang/ThreadGroup/NullThreadName.java fails with Thread group is no,t destroyed ,fastdebug LINUX

2013-02-12 Thread Seán Coffey
Approved, regards, Sean. On 12/02/2013 10:00, Chris Hegarty wrote: This is a request for approval to backport a test stabilization change from JDK8 to 7u-dev. http://bugs.sun.com/view_bug.do?bug_id=6776144 JDK8 changeset: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/7eb0debca9b3 JDK8 review

Re: [7u-next] Request for approval for CR 6963841 - java/util/concurrent/Phaser/Basic.java fails intermittently

2013-02-12 Thread Seán Coffey
Approved, regards, Sean. On 12/02/2013 09:54, Chris Hegarty wrote: This is a request for approval to backport a test stabilization change from JDK8 to 7u-dev. http://bugs.sun.com/view_bug.do?bug_id=6963841 JDK8 changeset: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/85b7fcf14249 JDK8 review

[7u-next] Request for approval for CR 7073295 - TEST_BUG: test/java/lang/instrument/ManifestTest.sh causing havoc (win)

2013-02-12 Thread Chris Hegarty
This is a request for approval to backport a test stabilization change from JDK8 to 7u-dev. http://bugs.sun.com/view_bug.do?bug_id=7073295 JDK8 changeset: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/26fe74aa48ef JDK8 review thread: http://mail.openjdk.java.net/pipermail/serviceability-dev/20

Re: [7u-next] Request for approval for CR 7032247 - java/net/InetAddress/GetLocalHostWithSM.java fails if hostname resolves, to loopback address

2013-02-12 Thread Seán Coffey
Approved, regards, Sean. On 12/02/2013 09:51, Chris Hegarty wrote: This is a request for approval to backport a test stabilization change from JDK8 to 7u-dev. http://bugs.sun.com/view_bug.do?bug_id=7032247 JDK8 changeset: http://hg.openjdk.java.net/jdk8/build/jdk/rev/d5d24c08f0dc JDK8 rev

[7u-next] Request for approval for CR 7044870 - java/nio/channels/DatagramChannel/SelectWhenRefused.java failed on SUSE Linux 10

2013-02-12 Thread Chris Hegarty
This is a request for approval to backport a test stabilization change from JDK8 to 7u-dev. http://bugs.sun.com/view_bug.do?bug_id=7044870 JDK8 changeset: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/c9e78769c0da JDK8 review thread: http://mail.openjdk.java.net/pipermail/nio-dev/2011-August/

hg: jdk7u/jdk7u-dev/jdk: 8007536: Incorrect copyright header in JDP files

2013-02-12 Thread dmitry . samersoff
Changeset: c9bc97a00f69 Author:dsamersoff Date: 2013-02-12 14:03 +0400 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/c9bc97a00f69 8007536: Incorrect copyright header in JDP files Summary: Copyright header in JDP files missed the "classpath exception" rule. Reviewed-by: mik

[7u-next] Request for approval for CR 6776144 - java/lang/ThreadGroup/NullThreadName.java fails with Thread group is no,t destroyed ,fastdebug LINUX

2013-02-12 Thread Chris Hegarty
This is a request for approval to backport a test stabilization change from JDK8 to 7u-dev. http://bugs.sun.com/view_bug.do?bug_id=6776144 JDK8 changeset: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/7eb0debca9b3 JDK8 review thread: http://mail.openjdk.java.net/pipermail/core-libs-dev/2011-No

[7u-next] Request for approval for CR 6963841 - java/util/concurrent/Phaser/Basic.java fails intermittently

2013-02-12 Thread Chris Hegarty
This is a request for approval to backport a test stabilization change from JDK8 to 7u-dev. http://bugs.sun.com/view_bug.do?bug_id=6963841 JDK8 changeset: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/85b7fcf14249 JDK8 review thread: http://mail.openjdk.java.net/pipermail/core-libs-dev/2012-Ap

[7u-next] Request for approval for CR 7032247 - java/net/InetAddress/GetLocalHostWithSM.java fails if hostname resolves, to loopback address

2013-02-12 Thread Chris Hegarty
This is a request for approval to backport a test stabilization change from JDK8 to 7u-dev. http://bugs.sun.com/view_bug.do?bug_id=7032247 JDK8 changeset: http://hg.openjdk.java.net/jdk8/build/jdk/rev/d5d24c08f0dc JDK8 review thread: http://mail.openjdk.java.net/pipermail/net-dev/2012-Sept

Re: [7u] Request for approval for 8007761: NTLM coding errors

2013-02-12 Thread Seán Coffey
Approved, regards, Sean. On 11/02/2013 02:16, Weijun Wang wrote: Hi All This is a request to backport a jdk8 fix into jdk7u. http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=8007761 Description. The fix is already included in jdk8 as: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/57cb

Re: [7u-dev] Request for approval for JDK-7175464 - entrySetView field is never updated in NavigableSubMap

2013-02-12 Thread Peter Levart
Hi Martin, It seems that adding a default method Comparator.reverseOrder() had an impact on the code in Collections. In he following code in Collections: private static class ReverseComparator implements Comparator>, Serializable { private static final long serialVersionU

7u14 Request for approval for CR 8006691 - Remove jvm_version_info.is_kernel_jvm field

2013-02-12 Thread Zhengyu Gu
This is a simple change to remove is_kernel_jvm field in jvm_version_info, as kernel VM has been deprecated. Bug: http://bugs.sun.com/view_bug.do?bug_id=8006691 Webrev: http://cr.openjdk.java.net/~zgu/8006691/jdk8/webrev.00/ It has been reviewed by mchung Thanks, -Zhengyu

Re: [7u] Request for approval for 8007536 - Incorrect copyright header in JDP files

2013-02-12 Thread Seán Coffey
Approved, regards, Sean. On 11/02/2013 14:57, Dmitry Samersoff wrote: Requesting approval to push the following change to jdk7u: Summary Copyright header in JDP files missed the "classpath exception" rule. webrev: http://cr.openjdk.java.net/~dsamersoff/8007536.JDP-COPYRIGHT/webrev.01/jdk/

Re: [7u] review and approval request for 8007515: update RMI tests on ProblemList.txt

2013-02-12 Thread Alan Bateman
On 11/02/2013 23:02, Stuart Marks wrote: I thought about dealing with TEST.ROOT in a separate bug but that just seems like busy-work. Updating the problem list is at least as much about making sure that the testing configurations between jdk8 and jdk7u are brought closer together as much as i