Re: 7u40: Request for approval for CR 8013434 - Xalan and Xerces internal ObjectFactory need rework

2013-06-11 Thread Seán Coffey
Thanks for the background information Joe. Can you run this by the CCC team then ? (log a request for jdk7u) regards, Sean. On 10/06/2013 23:13, huizhe wang wrote: Hi Sean, It was on my request that Daniel took the trouble to make the backports on this and several of his previous JDK8 change

Re: [7u] Request for approval for (jaxp): 8016153 : Property http://javax.xml.XMLConstants/property/accessExternalDTD is not recognized.

2013-06-11 Thread Seán Coffey
Approved. The jaxp XSLTC tests found this issue. Shouldn't you add the noreg-sqe label with a comment to that effect ? regards, Sean. On 10/06/2013 22:50, huizhe wang wrote: Hi, This is a request for approval to backport 8016153 to JDK7. This is a fix to the JAXP 1.5 upgrade so that 3rd par

Re: 7u40: Request for approval for CR 8013434 - Xalan and Xerces internal ObjectFactory need rework

2013-06-11 Thread Daniel Fuchs
Hi Seán, On 6/11/13 10:31 AM, Seán Coffey wrote: Thanks for the background information Joe. Can you run this by the CCC team then ? (log a request for jdk7u) In the CCC I had mentioned our intent to backport the fix to jdk7u-dev - but maybe that went unnoticed... -- daniel regards, Sean.

Re: [7u40] Request for approval : 8000450: Restrict access to com/sun/corba/se/impl package

2013-06-11 Thread Dalibor Topic
On 6/10/13 8:56 PM, Seán Coffey wrote: > I'd like to backport this fix to the jdk7u-dev repo. The changeset is > identical to that of jdk8. > > bug report : http://bugs.sun.com/view_bug.do?bug_id=8000450 > JDK 8 webrev : http://cr.openjdk.java.net/~coffeys/webrev.8000450.jdk8/ > JDK 8 review thre

Re: 7u40: Request for approval for CR 8013434 - Xalan and Xerces internal ObjectFactory need rework

2013-06-11 Thread Seán Coffey
Daniel, I just see that now. My understanding is that the "release" field in CCC should specify the earliest fix release which the change is planned for. I'm cc'ing Joe Darcy for advice. Joe - is this change ok for JDK 7u integration ? regards, Sean. On 11/06/2013 10:10, Daniel Fuchs wrote:

hg: jdk7u/jdk7u-dev/jdk: 8000450: Restrict access to com/sun/corba/se/impl package

2013-06-11 Thread sean . coffey
Changeset: 6417b1a98b80 Author:coffeys Date: 2013-06-06 14:10 +0100 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/6417b1a98b80 8000450: Restrict access to com/sun/corba/se/impl package Reviewed-by: alanb, chegar, lancea ! src/share/lib/security/java.security-linux ! src/s

[7u] Rquest for approval for CR 7184908 TEST_BUG: [macosx] closed/com/sun/java/swing/plaf/gtk/4928019/bug4928019.java fails

2013-06-11 Thread Konstantin Shefov
Hello, Please approve a direct backport from jdk 8 for the fix of the issue: 7184908 TEST_BUG: [macosx] closed/com/sun/java/swing/plaf/gtk/4928019/bug4928019.java fails Test bug fix. Move from closed to openjdk repo. http://bugs.sun.com/view_bug.do?bug_id=7184908 The webrev is: http://cr.op

Re: Review request for JDK-7179353: try-with-resources fails to compile with generic exception parameters

2013-06-11 Thread Vicente-Arturo Romero-Zaldivar
On 10/06/13 11:22, Andrew Hughes wrote: - Original Message - Hi, This is the backport to jdk7_dev of the the patch already pushed to jdk8/tl/langtools (changeset 9f11c7676cd5). You can find the public webrev at: http://cr.openjdk.java.net/~vromero/7179353_jdk7/webrev.00/ Thanks, Vice

Re: [7u] Rquest for approval for CR 7184908 TEST_BUG: [macosx] closed/com/sun/java/swing/plaf/gtk/4928019/bug4928019.java fails

2013-06-11 Thread Seán Coffey
Approved. regards, Sean. On 11/06/2013 11:27, Konstantin Shefov wrote: Hello, Please approve a direct backport from jdk 8 for the fix of the issue: 7184908 TEST_BUG: [macosx] closed/com/sun/java/swing/plaf/gtk/4928019/bug4928019.java fails Test bug fix. Move from closed to openjdk repo. h

hg: jdk7u/jdk7u-dev/jdk: 7184908: TEST_BUG: [macosx] closed/com/sun/java/swing/plaf/gtk/4928019/bug4928019.java fails

2013-06-11 Thread konstantin . shefov
Changeset: d5a64b1a77b1 Author:kshefov Date: 2013-06-11 15:34 +0400 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/d5a64b1a77b1 7184908: TEST_BUG: [macosx] closed/com/sun/java/swing/plaf/gtk/4928019/bug4928019.java fails Reviewed-by: alexsch, serb + test/com/sun/java/swin

[7u communication] RDP2 milestone for 7u40 approaching.

2013-06-11 Thread Seán Coffey
Just a reminder to all that the PIT code freeze for 7u40 b30 takes place tomorrow (Wednesday 12th) at 10am Pacific Time. The HSX schedule will differ slightly. Please plan on having all 7u40 targeted fixes pushed to jdk7u-dev forest by then. Once 7u40 b30 is promoted next week, the rampdown p

[7u40] Request for approval : 7164256 : EnumMap clone doesn't clear the entrySet keeping a reference to the original Map

2013-06-11 Thread Seán Coffey
I'd like to port this back to 7u. Changeset applies without issue. bug report : http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7164256 jdk8 changeset : http://hg.openjdk.java.net/jdk8/tl/jdk/rev/757a5129fad7 JDK 8 review thread : http://mail.openjdk.java.net/pipermail/core-libs-dev/2012-May/

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

2013-06-11 Thread rob . mckenna
Changeset: 6e8c218b8eb8 Author:robm Date: 2013-05-28 16:35 +0100 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/6e8c218b8eb8 7038105: File.isHidden() should return true for pagefile.sys and hiberfil.sys Reviewed-by: alanb ! src/windows/native/java/io/WinNTFileSystem_md.c !

Re: [7u] Request for approval for (jaxp): 8016153 : Property http://javax.xml.XMLConstants/property/accessExternalDTD is not recognized.

2013-06-11 Thread huizhe wang
I can add noreg-sqe. But I attached a test that can be run manually since it requires a 3rd party jar file (e.g. Xerces). Would you think reg-manual is correct? Regards, Joe On 6/11/2013 1:52 AM, Seán Coffey wrote: Approved. The jaxp XSLTC tests found this issue. Shouldn't you add the noreg-s

Re: [7u] Request for approval for (jaxp): 8016153 : Property http://javax.xml.XMLConstants/property/accessExternalDTD is not recognized.

2013-06-11 Thread Seán Coffey
I think the QA team use reg-manual label for their records. In general I've seen the noreg-* used for openJDK contributions where a jtreg testcase isn't provided : http://openjdk.java.net/guide/changePlanning.html Maybe noreg-external is more appropriate for your case. regards, Sean. On 11/0

[7u40] Request for approval : 8013827 & 8011950

2013-06-11 Thread Seán Coffey
I'd like to port this recent jdk8 fix to 7u-dev so that it makes the upcoming 7u40 release. Changes apply cleanly to jdk7u base and tests look ok. 8013827: File.createTempFile hangs with temp file starting with 'com1.4' 8011950: java.io.File.createTempFile enters infinite loop when passed inva

Re: [7u40] Request for approval : 7164256 : EnumMap clone doesn't clear the entrySet keeping a reference to the original Map

2013-06-11 Thread Dalibor Topic
On 6/11/13 4:45 PM, Seán Coffey wrote: > I'd like to port this back to 7u. Changeset applies without issue. > > bug report : http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7164256 > jdk8 changeset : http://hg.openjdk.java.net/jdk8/tl/jdk/rev/757a5129fad7 > JDK 8 review thread : > http://mail.

Re: [7u] Request for approval for (jaxp): 8016153 : Property http://javax.xml.XMLConstants/property/accessExternalDTD is not recognized.

2013-06-11 Thread huizhe wang
Make sense. Updated. Thanks! Joe On 6/11/2013 10:27 AM, Seán Coffey wrote: I think the QA team use reg-manual label for their records. In general I've seen the noreg-* used for openJDK contributions where a jtreg testcase isn't provided : http://openjdk.java.net/guide/changePlanning.html

Re: [7u40] Request for approval : 8013827 & 8011950

2013-06-11 Thread Dalibor Topic
On 6/11/13 7:57 PM, Seán Coffey wrote: > I'd like to port this recent jdk8 fix to 7u-dev so that it makes the upcoming > 7u40 release. Changes apply cleanly to jdk7u base and tests look ok. > > 8013827: File.createTempFile hangs with temp file starting with 'com1.4' > 8011950: java.io.File.create

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

2013-06-11 Thread sean . coffey
Changeset: 80fefc247e3d Author:alanb Date: 2012-06-07 12:31 +0100 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/80fefc247e3d 7164256: EnumMap clone doesn't clear the entrySet keeping a reference to the original Map Reviewed-by: alanb, chegar, forax, mduigou Contributed-by

hg: jdk7u/jdk7u-dev/jaxp: 8016153: Property http://javax.xml.XMLConstants/property/accessExternalDTD is not recognized.

2013-06-11 Thread huizhe . wang
Changeset: e0d70a8119e8 Author:joehw Date: 2013-06-11 14:22 -0700 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jaxp/rev/e0d70a8119e8 8016153: Property http://javax.xml.XMLConstants/property/accessExternalDTD is not recognized. Reviewed-by: lancea, dfuchs ! src/com/sun/org/apach

hg: jdk7u/jdk7u-dev/hotspot: 9 new changesets

2013-06-11 Thread alejandro . murillo
Changeset: e341c460dcbf Author:johnc Date: 2013-05-31 12:18 -0700 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/hotspot/rev/e341c460dcbf 7176479: G1: JVM crashes on T5-8 system with 1.5 TB heap Summary: Refactor G1's hot card cache and card counts table into their own files. Simp

jdk7u40-b29: HotSpot

2013-06-11 Thread Alejandro E Murillo
hs24-b48 has been integrated into jdk7u40-b29. http://hg.openjdk.java.net/jdk7u/jdk7u/rev/dbe42a627813 http://hg.openjdk.java.net/jdk7u/jdk7u/corba/rev/1ad0a5a3f7b1 http://hg.openjdk.java.net/jdk7u/jdk7u/hotspot/rev/d74376b0f20b http://hg.openjdk.java.net/jdk7u/jdk7u/jaxp/rev/740942c76fb6 http://

hg: jdk7u/jdk7u/hotspot: 9 new changesets

2013-06-11 Thread alejandro . murillo
Changeset: e341c460dcbf Author:johnc Date: 2013-05-31 12:18 -0700 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/hotspot/rev/e341c460dcbf 7176479: G1: JVM crashes on T5-8 system with 1.5 TB heap Summary: Refactor G1's hot card cache and card counts table into their own files. Simplify

Re: [7u communication] RDP2 milestone for 7u40 approaching.

2013-06-11 Thread Alejandro E Murillo
The corresponding deadline for Hotspot changes going to hsx24/7u40 is Thursday (June 13) at 6AM PT, as we plan to get the hs24 snapshot that day, run PIT over the weekend and push next Tuesday if PIT goes well. Thanks, Alejandro On 6/11/2013 8:35 AM, Seán Coffey wrote: Just a reminder to