From masayoshi.okutsu at oracle.com Mon Mar 2 09:34:02 2015 From: masayoshi.okutsu at oracle.com (Masayoshi Okutsu) Date: Mon, 02 Mar 2015 18:34:02 +0900 Subject: [8u60] Request for approval: 8072602: Unpredictable timezone on Windows when OS's timezone is not found in tzmappings Message-ID: <54F42E8A.2000808@oracle.com> Hello, Could you please approve backport of the fix for 8072602 to jdk8u-dev? The fix is identical to the JDK 9 one. Bug: https://bugs.openjdk.java.net/browse/JDK-8072602 Webrev: http://cr.openjdk.java.net/~okutsu/8u/8072602/webrev.00/ JDK9 changeset: http://hg.openjdk.java.net/jdk9/dev/jdk/rev/e1f68774d811 Thanks, Masayoshi From sean.coffey at oracle.com Mon Mar 2 09:38:34 2015 From: sean.coffey at oracle.com (=?UTF-8?B?U2XDoW4gQ29mZmV5?=) Date: Mon, 02 Mar 2015 09:38:34 +0000 Subject: [8u60] Request for approval: 8072602: Unpredictable timezone on Windows when OS's timezone is not found in tzmappings In-Reply-To: <54F42E8A.2000808@oracle.com> References: <54F42E8A.2000808@oracle.com> Message-ID: <54F42F9A.7040600@oracle.com> review thread : http://mail.openjdk.java.net/pipermail/i18n-dev/2015-February/001630.html Approved. regards, Sean. On 02/03/2015 09:34, Masayoshi Okutsu wrote: > Hello, > > Could you please approve backport of the fix for 8072602 to jdk8u-dev? > The fix is identical to the JDK 9 one. > > Bug: > https://bugs.openjdk.java.net/browse/JDK-8072602 > > Webrev: > http://cr.openjdk.java.net/~okutsu/8u/8072602/webrev.00/ > > JDK9 changeset: > http://hg.openjdk.java.net/jdk9/dev/jdk/rev/e1f68774d811 > > Thanks, > Masayoshi From rob.mckenna at oracle.com Mon Mar 2 15:34:28 2015 From: rob.mckenna at oracle.com (Rob McKenna) Date: Mon, 02 Mar 2015 15:34:28 +0000 Subject: [8u-communication] JDK 8u60 timeline Message-ID: <54F48304.50406@oracle.com> With the planned JDK 8u40 GA date [1] around the corner its time to start thinking about 8u60. I've published a proposal for the key milestone dates relating to the 8u60 release: http://openjdk.java.net/projects/jdk8u/releases/8u60.html December 2014 8u-dev forests begin collecting 8u60 fixes (completed) March 2015 Feature Complete May 2015 Rampdown 1 June 2015 RampDown 2 (Fork for the stabilization forests) September 2015 GA Note that these timelines are preliminary only and are subject to change. Please refer to the JDK 8 milestones definition page [2] for a summary of what each milestone consists of. Please submit objections by Friday the 6th of March 2015. Thanks, -Rob [1] http://openjdk.java.net/projects/jdk8u/releases/8u40.html [2]http://openjdk.java.net/projects/jdk8/milestones From sgehwolf at redhat.com Mon Mar 2 16:06:32 2015 From: sgehwolf at redhat.com (Severin Gehwolf) Date: Mon, 02 Mar 2015 17:06:32 +0100 Subject: [8u60] Request for approval to backport: 8067330: ZERO_ARCHDEF incorrectly defined for PPC/PPC64 architectures Message-ID: <1425312392.3461.19.camel@redhat.com> Hi, I've sent this email before[1], but would like for this backport to get into 8u60 before it closes this month[2]. The patch in question only touches Zero make variables and should be fairly safe to backport. The patch is the same (modulo autogen.sh generation timestamp) as for JDK 9. I'd also please need a sponsor who can push this change for me. Thanks in advance! Original 9 bug: https://bugs.openjdk.java.net/browse/JDK-8067330 webrev: http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-8067330/webrev.jdk8.01/ JDK 9 review thread: http://mail.openjdk.java.net/pipermail/build-dev/2014-December/013895.html Please let me know if there is anything else I need to do. Cheers, Severin [1] http://mail.openjdk.java.net/pipermail/build-dev/2015-February/014394.html [2] http://openjdk.java.net/projects/jdk8u/releases/8u60.html From sean.coffey at oracle.com Mon Mar 2 16:56:21 2015 From: sean.coffey at oracle.com (=?UTF-8?B?U2XDoW4gQ29mZmV5?=) Date: Mon, 02 Mar 2015 16:56:21 +0000 Subject: [8u60] Request for approval to backport: 8067330: ZERO_ARCHDEF incorrectly defined for PPC/PPC64 architectures In-Reply-To: <1425312392.3461.19.camel@redhat.com> References: <1425312392.3461.19.camel@redhat.com> Message-ID: <54F49635.8060101@oracle.com> Hi Severin, I'd missed your previous request. It was marked as a review request! Consider this approved for jdk8u-dev. Can you add a 'noreg-build' label to the bug report ? Erik, would you be willing to push this changeset to the 8u-dev forest ? Looks like there's some closed code that needs updating also. regards, Sean. On 02/03/15 16:06, Severin Gehwolf wrote: > Hi, > > I've sent this email before[1], but would like for this backport to get > into 8u60 before it closes this month[2]. The patch in question only > touches Zero make variables and should be fairly safe to backport. The > patch is the same (modulo autogen.sh generation timestamp) as for JDK 9. > I'd also please need a sponsor who can push this change for me. > > Thanks in advance! > > Original 9 bug: https://bugs.openjdk.java.net/browse/JDK-8067330 > webrev: > http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-8067330/webrev.jdk8.01/ > > JDK 9 review thread: > http://mail.openjdk.java.net/pipermail/build-dev/2014-December/013895.html > > Please let me know if there is anything else I need to do. > > Cheers, > Severin > > [1] > http://mail.openjdk.java.net/pipermail/build-dev/2015-February/014394.html > [2] http://openjdk.java.net/projects/jdk8u/releases/8u60.html > From sgehwolf at redhat.com Mon Mar 2 17:00:36 2015 From: sgehwolf at redhat.com (Severin Gehwolf) Date: Mon, 02 Mar 2015 18:00:36 +0100 Subject: [8u60] Request for approval to backport: 8067330: ZERO_ARCHDEF incorrectly defined for PPC/PPC64 architectures In-Reply-To: <54F49635.8060101@oracle.com> References: <1425312392.3461.19.camel@redhat.com> <54F49635.8060101@oracle.com> Message-ID: <1425315636.3461.28.camel@redhat.com> Hi, On Mon, 2015-03-02 at 16:56 +0000, Se?n Coffey wrote: > Hi Severin, > > I'd missed your previous request. It was marked as a review request! Sorry, my bad. > Consider this approved for jdk8u-dev. Can you add a 'noreg-build' label > to the bug report ? Done. > Erik, would you be willing to push this changeset to the 8u-dev forest > ? Looks like there's some closed code that needs updating also. Many thanks! Cheers, Severin > regards, > Sean. > > On 02/03/15 16:06, Severin Gehwolf wrote: > > Hi, > > > > I've sent this email before[1], but would like for this backport to get > > into 8u60 before it closes this month[2]. The patch in question only > > touches Zero make variables and should be fairly safe to backport. The > > patch is the same (modulo autogen.sh generation timestamp) as for JDK 9. > > I'd also please need a sponsor who can push this change for me. > > > > Thanks in advance! > > > > Original 9 bug: https://bugs.openjdk.java.net/browse/JDK-8067330 > > webrev: > > http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-8067330/webrev.jdk8.01/ > > > > JDK 9 review thread: > > http://mail.openjdk.java.net/pipermail/build-dev/2014-December/013895.html > > > > Please let me know if there is anything else I need to do. > > > > Cheers, > > Severin > > > > [1] > > http://mail.openjdk.java.net/pipermail/build-dev/2015-February/014394.html > > [2] http://openjdk.java.net/projects/jdk8u/releases/8u60.html > > > From daniel.fuchs at oracle.com Mon Mar 2 18:32:46 2015 From: daniel.fuchs at oracle.com (Daniel Fuchs) Date: Mon, 02 Mar 2015 19:32:46 +0100 Subject: [8u60] Request for review & approval: 8074125: Add SerializedLogRecord test to jdk8-u Message-ID: <54F4ACCE.1010505@oracle.com> Hi, This is a request for review & approval to push a new test to jdk8-u. This test has already been pushed in JDK 9 as part of a bigger changeset: JDK-8072645. To verify the serial compatibility of LogRecord between JDK 8 and JDK 9, the SerializedLogRecord.java should also be pushed to JDK 8, unchanged. JBS issue: https://bugs.openjdk.java.net/browse/JDK-8074125 8u webrev: http://cr.openjdk.java.net/~dfuchs/webrev_8074125/webrev.00/ JDK 9 changeset that contained the test (if that's helpful): http://hg.openjdk.java.net/jdk9/dev/jdk/file/710db3eff010/test/java/util/logging/HigherResolutionTimeStamps/SerializeLogRecord.java JDK 9 review thread of the JDK 9 changeset which contained the test, if that's helpful: http://mail.openjdk.java.net/pipermail/core-libs-dev/2015-February/031486.html best regards, -- daniel From sean.coffey at oracle.com Mon Mar 2 21:15:54 2015 From: sean.coffey at oracle.com (=?UTF-8?B?U2XDoW4gQ29mZmV5?=) Date: Mon, 02 Mar 2015 21:15:54 +0000 Subject: [8u60] Request for review & approval: 8074125: Add SerializedLogRecord test to jdk8-u In-Reply-To: <54F4ACCE.1010505@oracle.com> References: <54F4ACCE.1010505@oracle.com> Message-ID: <54F4D30A.5060609@oracle.com> Looks good to me Daniel. I've leave you decide if you want to replace the @bug tag with the 8u only bug number. Approved. regards, Sean. On 02/03/2015 18:32, Daniel Fuchs wrote: > Hi, > > This is a request for review & approval to push a new test to jdk8-u. > > This test has already been pushed in JDK 9 as part of a bigger > changeset: JDK-8072645. > To verify the serial compatibility of LogRecord between JDK 8 and JDK 9, > the SerializedLogRecord.java should also be pushed to JDK 8, unchanged. > > JBS issue: > https://bugs.openjdk.java.net/browse/JDK-8074125 > 8u webrev: > http://cr.openjdk.java.net/~dfuchs/webrev_8074125/webrev.00/ > > JDK 9 changeset that contained the test (if that's helpful): > http://hg.openjdk.java.net/jdk9/dev/jdk/file/710db3eff010/test/java/util/logging/HigherResolutionTimeStamps/SerializeLogRecord.java > > > JDK 9 review thread of the JDK 9 changeset which contained the test, > if that's helpful: > http://mail.openjdk.java.net/pipermail/core-libs-dev/2015-February/031486.html > > > best regards, > > -- daniel From erik.joelsson at oracle.com Tue Mar 3 09:12:31 2015 From: erik.joelsson at oracle.com (Erik Joelsson) Date: Tue, 03 Mar 2015 10:12:31 +0100 Subject: [8u60] Request for approval to backport: 8067330: ZERO_ARCHDEF incorrectly defined for PPC/PPC64 architectures In-Reply-To: <54F49635.8060101@oracle.com> References: <1425312392.3461.19.camel@redhat.com> <54F49635.8060101@oracle.com> Message-ID: <54F57AFF.70508@oracle.com> Sure, I can do it. /Erik On 2015-03-02 17:56, Se?n Coffey wrote: > Hi Severin, > > I'd missed your previous request. It was marked as a review request! > Consider this approved for jdk8u-dev. Can you add a 'noreg-build' > label to the bug report ? > > Erik, would you be willing to push this changeset to the 8u-dev > forest ? Looks like there's some closed code that needs updating also. > > regards, > Sean. > > On 02/03/15 16:06, Severin Gehwolf wrote: >> Hi, >> >> I've sent this email before[1], but would like for this backport to get >> into 8u60 before it closes this month[2]. The patch in question only >> touches Zero make variables and should be fairly safe to backport. The >> patch is the same (modulo autogen.sh generation timestamp) as for JDK 9. >> I'd also please need a sponsor who can push this change for me. >> >> Thanks in advance! >> >> Original 9 bug: https://bugs.openjdk.java.net/browse/JDK-8067330 >> webrev: >> http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-8067330/webrev.jdk8.01/ >> >> JDK 9 review thread: >> http://mail.openjdk.java.net/pipermail/build-dev/2014-December/013895.html >> >> >> Please let me know if there is anything else I need to do. >> >> Cheers, >> Severin >> >> [1] >> http://mail.openjdk.java.net/pipermail/build-dev/2015-February/014394.html >> >> [2] http://openjdk.java.net/projects/jdk8u/releases/8u60.html >> > From daniel.fuchs at oracle.com Tue Mar 3 13:57:57 2015 From: daniel.fuchs at oracle.com (Daniel Fuchs) Date: Tue, 03 Mar 2015 14:57:57 +0100 Subject: [8u60] Request for review & approval: 8074125: Add SerializedLogRecord test to jdk8-u In-Reply-To: <54F4D30A.5060609@oracle.com> References: <54F4ACCE.1010505@oracle.com> <54F4D30A.5060609@oracle.com> Message-ID: <54F5BDE5.3000607@oracle.com> On 02/03/15 22:15, Se?n Coffey wrote: > Looks good to me Daniel. I've leave you decide if you want to replace > the @bug tag with the 8u only bug number. Thanks Se?n. After thinking this over I believe using the the jdk8u bug number might be better, as it explains why this test was added to jdk 8. Will change that before pushing! -- daniel > > Approved. > > regards, > Sean. > > On 02/03/2015 18:32, Daniel Fuchs wrote: >> Hi, >> >> This is a request for review & approval to push a new test to jdk8-u. >> >> This test has already been pushed in JDK 9 as part of a bigger >> changeset: JDK-8072645. >> To verify the serial compatibility of LogRecord between JDK 8 and JDK 9, >> the SerializedLogRecord.java should also be pushed to JDK 8, unchanged. >> >> JBS issue: >> https://bugs.openjdk.java.net/browse/JDK-8074125 >> 8u webrev: >> http://cr.openjdk.java.net/~dfuchs/webrev_8074125/webrev.00/ >> >> JDK 9 changeset that contained the test (if that's helpful): >> http://hg.openjdk.java.net/jdk9/dev/jdk/file/710db3eff010/test/java/util/logging/HigherResolutionTimeStamps/SerializeLogRecord.java >> >> >> JDK 9 review thread of the JDK 9 changeset which contained the test, >> if that's helpful: >> http://mail.openjdk.java.net/pipermail/core-libs-dev/2015-February/031486.html >> >> >> best regards, >> >> -- daniel > From sean.coffey at oracle.com Tue Mar 3 20:21:26 2015 From: sean.coffey at oracle.com (=?UTF-8?B?U2XDoW4gQ29mZmV5?=) Date: Tue, 03 Mar 2015 20:21:26 +0000 Subject: [8u-communication] JDK 8u40 released today! Message-ID: <54F617C6.2030906@oracle.com> I'd like to announce that JDK 8u40 has become available for download today [0]. Thanks to all those who have contributed towards it. OpenJDK 8u40 source code is available via the 8u40 stabilization forest [1]. I plan to update the OpenJDK 8u project page with latest status and to also ask the OpenJDK ops team to mark the 8u40 forests as read-only. If you're packaging this release, it could be useful to let subscribed members know about it via communication on this mailing list. Please continue to contribute fixes back to the jdk8u-dev forest [2] which is already gathering changes for the next non-CPU JDK8u release. Regards, Sean. [0] http://www.oracle.com/technetwork/java/javase/downloads/index.html [1] http://hg.openjdk.java.net/jdk8u/jdk8u40/ [2] http://hg.openjdk.java.net/jdk8u/jdk8u-dev From alejandro.murillo at oracle.com Tue Mar 3 21:17:23 2015 From: alejandro.murillo at oracle.com (Alejandro E Murillo) Date: Tue, 03 Mar 2015 14:17:23 -0700 Subject: jdk8u60-b05: HotSpot Message-ID: <54F624E3.4020204@oracle.com> hs25.60-b05 has been integrated into jdk8u60-b05. http://hg.openjdk.java.net/jdk8u/jdk8u/rev/e532f3672f63 http://hg.openjdk.java.net/jdk8u/jdk8u/corba/rev/983825f68350 http://hg.openjdk.java.net/jdk8u/jdk8u/hotspot/rev/b13f1890afb8 http://hg.openjdk.java.net/jdk8u/jdk8u/jaxp/rev/13a5799e90e9 http://hg.openjdk.java.net/jdk8u/jdk8u/jaxws/rev/4dfd5dbd3014 http://hg.openjdk.java.net/jdk8u/jdk8u/jdk/rev/433942aab113 http://hg.openjdk.java.net/jdk8u/jdk8u/langtools/rev/fc98314cff57 http://hg.openjdk.java.net/jdk8u/jdk8u/nashorn/rev/6f44964fbab3 Component : VM Status : 0 major failures, 0 minor failures Date : 02/27/2015 at 21:00 MSK Tested By : STT_VM Cost(total man-days): 0.3 Workspace : 2015-02-27-174202.amurillo.hs25-60-b05-snapshot Bundles : 2015-02-27-174202.amurillo.hs25-60-b05-snapshot Platforms : Others Tests : Log : link Browsers : NA Patches : NA Number of Tests Executed : 391552 passed tests, 3775 failed tests (no new failures) Bug verification status: ====================================== Tested, Pass: Tested, Pass (partial fixes): Tested, Fail: Untested bug fixes: 8064335: Null pointer dereference in hotspot/src/share/vm/classfile/verifier.cpp 8064815: Zero+PPC64: Stack overflow when running Maven 8067231: Zero builds fails after JDK-6898462 8067331: Zero: Atomic::xchg and Atomic::xchg_ptr need full memory barrier 8072383: resolve conflicts between open and closed ports 8073514: new hotspot build - hs25.60-b05 New bugs filed: Bugs in PIT build: Bugs in earlier promoted build: Number of PIT requested: 1 Integration target J2SE build number: jdk8u60-b05 Issues and Notes: This is PIT for HS25.60-b05 for jdk8u60-b05. Go for integration. -- Alejandro From kumar.x.srinivasan at oracle.com Tue Mar 3 21:19:27 2015 From: kumar.x.srinivasan at oracle.com (Kumar Srinivasan) Date: Tue, 03 Mar 2015 13:19:27 -0800 Subject: [8u-dev]; Request approval for 8073972: Deprecate Multi-Version Java Launcher (mJRE) for JDK8 Message-ID: <54F6255F.8080506@oracle.com> Hello, Please approve integration into 8u-dev, final destination being 8u60. Webrev: http://cr.openjdk.java.net/~ksrini/8073972/webrev.00/ Bug: http://bugs.openjdk.java.net/browse/JDK-8073972 Discussion: http://mail.openjdk.java.net/pipermail/core-libs-dev/2015-March/032013.html Thanks Kumar From iris.clark at oracle.com Wed Mar 4 06:21:44 2015 From: iris.clark at oracle.com (Iris Clark) Date: Tue, 3 Mar 2015 22:21:44 -0800 (PST) Subject: Java SE 8 Maintenance Release (JSR 337) posted to jcp.org Message-ID: <95129e3e-7a62-4dab-a2dc-81f52ee560f9@default> The Maintenance Release of JSR 337 which includes an updated specification, RI, and TCK is available here: https://jcp.org/aboutJava/communityprocess/mrel/jsr337/index.html Thanks, iris From sean.coffey at oracle.com Wed Mar 4 08:32:29 2015 From: sean.coffey at oracle.com (=?UTF-8?B?U2XDoW4gQ29mZmV5?=) Date: Wed, 04 Mar 2015 08:32:29 +0000 Subject: [8u-dev]; Request approval for 8073972: Deprecate Multi-Version Java Launcher (mJRE) for JDK8 In-Reply-To: <54F6255F.8080506@oracle.com> References: <54F6255F.8080506@oracle.com> Message-ID: <54F6C31D.40109@oracle.com> Approved. regards, Sean. On 03/03/2015 21:19, Kumar Srinivasan wrote: > Hello, > > Please approve integration into 8u-dev, final destination being 8u60. > > Webrev: > http://cr.openjdk.java.net/~ksrini/8073972/webrev.00/ > > Bug: > http://bugs.openjdk.java.net/browse/JDK-8073972 > > Discussion: > http://mail.openjdk.java.net/pipermail/core-libs-dev/2015-March/032013.html > > > Thanks > Kumar > From alexandr.scherbatiy at oracle.com Wed Mar 4 09:05:21 2015 From: alexandr.scherbatiy at oracle.com (Alexander Scherbatiy) Date: Wed, 04 Mar 2015 12:05:21 +0300 Subject: [8u60] Request for approval: 8068040 [macosx] Combo box consuming ENTER key events Message-ID: <54F6CAD1.9000305@oracle.com> Hello, Could you approve the direct backport of the fix to JDK 8u-dev. The bug: https://bugs.openjdk.java.net/browse/JDK-8068040 The webrev: http://cr.openjdk.java.net/~alexsch/8068040/webrev.01/ The review thread: http://mail.openjdk.java.net/pipermail/swing-dev/2015-January/004130.html The JDK 9 changeset: http://hg.openjdk.java.net/jdk9/client/jdk/rev/772a6e0e14fe Thanks, Alexandr. From sean.coffey at oracle.com Wed Mar 4 10:17:59 2015 From: sean.coffey at oracle.com (=?windows-1252?Q?Se=E1n_Coffey?=) Date: Wed, 04 Mar 2015 10:17:59 +0000 Subject: [8u60] Request for approval: 8068040 [macosx] Combo box consuming ENTER key events In-Reply-To: <54F6CAD1.9000305@oracle.com> References: <54F6CAD1.9000305@oracle.com> Message-ID: <54F6DBD7.2010306@oracle.com> Please add a noreg- label to the bug report. Approved. regards, Sean. On 04/03/15 09:05, Alexander Scherbatiy wrote: > > Hello, > > Could you approve the direct backport of the fix to JDK 8u-dev. > > The bug: https://bugs.openjdk.java.net/browse/JDK-8068040 > The webrev: http://cr.openjdk.java.net/~alexsch/8068040/webrev.01/ > The review thread: > http://mail.openjdk.java.net/pipermail/swing-dev/2015-January/004130.html > The JDK 9 changeset: > http://hg.openjdk.java.net/jdk9/client/jdk/rev/772a6e0e14fe > > Thanks, > Alexandr. > From sean.coffey at oracle.com Wed Mar 4 11:08:51 2015 From: sean.coffey at oracle.com (=?UTF-8?B?U2XDoW4gQ29mZmV5?=) Date: Wed, 04 Mar 2015 11:08:51 +0000 Subject: [8u-dev] Request for approval : 8067105 : Socket returned by ServerSocket.accept() is inherited by child process on Windows Message-ID: <54F6E7C3.20705@oracle.com> Looking to backport this fix to jdk8u-dev. The jdk9 patch applies cleanly post module path changes. jdk9 changeset : http://hg.openjdk.java.net/jdk9/dev/jdk/rev/d4eae56c57ab review thread : http://mail.openjdk.java.net/pipermail/net-dev/2014-December/008789.html bug report : https://bugs.openjdk.java.net/browse/JDK-8067105 regards, Sean. From dalibor.topic at oracle.com Wed Mar 4 12:25:27 2015 From: dalibor.topic at oracle.com (dalibor topic) Date: Wed, 04 Mar 2015 13:25:27 +0100 Subject: [8u-dev] Request for approval : 8067105 : Socket returned by ServerSocket.accept() is inherited by child process on Windows In-Reply-To: <54F6E7C3.20705@oracle.com> References: <54F6E7C3.20705@oracle.com> Message-ID: <54F6F9B7.30408@oracle.com> Approved. On 04.03.2015 12:08, Se?n Coffey wrote: > Looking to backport this fix to jdk8u-dev. The jdk9 patch applies > cleanly post module path changes. > > jdk9 changeset : http://hg.openjdk.java.net/jdk9/dev/jdk/rev/d4eae56c57ab > review thread : > http://mail.openjdk.java.net/pipermail/net-dev/2014-December/008789.html > bug report : https://bugs.openjdk.java.net/browse/JDK-8067105 > > regards, > Sean. > -- Dalibor Topic | Principal Product Manager Phone: +494089091214 | Mobile: +491737185961 ORACLE Deutschland B.V. & Co. KG | K?hneh?fe 5 | 22761 Hamburg ORACLE Deutschland B.V. & Co. KG Hauptverwaltung: Riesstr. 25, D-80992 M?nchen Registergericht: Amtsgericht M?nchen, HRA 95603 Gesch?ftsf?hrer: J?rgen Kunz Komplement?rin: ORACLE Deutschland Verwaltung B.V. Hertogswetering 163/167, 3543 AS Utrecht, Niederlande Handelsregister der Handelskammer Midden-Niederlande, Nr. 30143697 Gesch?ftsf?hrer: Alexander van der Ven, Astrid Kepper, Val Maher Oracle is committed to developing practices and products that help protect the environment From sean.coffey at oracle.com Wed Mar 4 19:51:33 2015 From: sean.coffey at oracle.com (=?UTF-8?B?U2XDoW4gQ29mZmV5?=) Date: Wed, 04 Mar 2015 19:51:33 +0000 Subject: [8u-communication] 8u60 JEPs Message-ID: <54F76245.9090706@oracle.com> Feature planning for the 8u60 release is currently under way. Like the 8u40 release, 8u60 will use the JEP 2.0 Process [1] to cover the creation and maintenance of the set of features intended for delivery in 8u60. Monitoring and tracking of such JEPs can be done via : https://bugs.openjdk.java.net/issues/?jql=type%20%3D%20JEP%20and%20fixVersion%20%3D%208u60 regards, Sean. [1]http://cr.openjdk.java.net/~mr/jep/jep-2.0-02.html From fridrich.strba at suse.com Thu Mar 5 08:58:29 2015 From: fridrich.strba at suse.com (Fridrich Strba) Date: Thu, 05 Mar 2015 09:58:29 +0100 Subject: [8u-communication] JDK 8u40 released today! In-Reply-To: <54F617C6.2030906@oracle.com> References: <54F617C6.2030906@oracle.com> Message-ID: <54F81AB5.4030601@suse.com> -----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 Hello, On 03/03/15 21:21, Se?n Coffey wrote: > If you're packaging this release, it could be useful to let > subscribed members know about it via communication on this mailing > list. Yes, this code, corresponding to the tag jdk8u40-b25 was submitted to openSUSE 13.2 and openSUSE Factory/Tumbleweed as java-1_8_0-openjdk-1.8.0.40. So, yes, we package it at SUSE. Cheers Fridrich -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iEYEARECAAYFAlT4Gq4ACgkQu9a1imXPdA/Z6ACff1oRYR15Cz8yx62MXDhYXOm3 p50AnRYv7RjRFcEP32+oJ4pzUqpM0GPP =IW1s -----END PGP SIGNATURE----- From shanliang.jiang at oracle.com Thu Mar 5 14:19:09 2015 From: shanliang.jiang at oracle.com (shanliang) Date: Thu, 05 Mar 2015 15:19:09 +0100 Subject: [8u60] Request for approval to backport: 8073148: "The server has decided to close this client connection" repeated continuously In-Reply-To: <541AEDF5.1040901@oracle.com> References: <541AEDF5.1040901@oracle.com> Message-ID: <54F865DD.1090406@oracle.com> Hi, Please approve this "clean" backport of JDK-8073148 fix to 8u60. Thanks, Shanliang From shanliang.jiang at oracle.com Thu Mar 5 14:23:24 2015 From: shanliang.jiang at oracle.com (shanliang) Date: Thu, 05 Mar 2015 15:23:24 +0100 Subject: [8u60] Request for approval to backport: 8073148: "The server has decided to close this client connection" repeated continuously In-Reply-To: <54F865DD.1090406@oracle.com> References: <541AEDF5.1040901@oracle.com> <54F865DD.1090406@oracle.com> Message-ID: <54F866DC.4030302@oracle.com> shanliang wrote: > Hi, > > Please approve this "clean" backport of JDK-8073148 fix to 8u60. > > Thanks, > Shanliang bug: https://bugs.openjdk.java.net/browse/JDK-8073148 Changeset: http://hg.openjdk.java.net/jdk9/dev/jdk/rev/dd67910c07f4 Review thread: : http://mail.openjdk.java.net/pipermail/serviceability-dev/2015-March/016861.html From rob.mckenna at oracle.com Thu Mar 5 15:44:03 2015 From: rob.mckenna at oracle.com (Rob McKenna) Date: Thu, 05 Mar 2015 15:44:03 +0000 Subject: [8u60] Request for approval to backport: 8073148: "The server has decided to close this client connection" repeated continuously In-Reply-To: <54F866DC.4030302@oracle.com> References: <541AEDF5.1040901@oracle.com> <54F865DD.1090406@oracle.com> <54F866DC.4030302@oracle.com> Message-ID: <54F879C3.4060604@oracle.com> Approved. -Rob On 05/03/15 14:23, shanliang wrote: > shanliang wrote: >> Hi, >> >> Please approve this "clean" backport of JDK-8073148 fix to 8u60. >> >> Thanks, >> Shanliang > bug: https://bugs.openjdk.java.net/browse/JDK-8073148 > Changeset: http://hg.openjdk.java.net/jdk9/dev/jdk/rev/dd67910c07f4 > Review thread: : > http://mail.openjdk.java.net/pipermail/serviceability-dev/2015-March/016861.html > > From jaroslav.bachorik at oracle.com Thu Mar 5 16:57:06 2015 From: jaroslav.bachorik at oracle.com (Jaroslav Bachorik) Date: Thu, 05 Mar 2015 17:57:06 +0100 Subject: [8u-dev] Request for approval: 8030115 - [parfait] warnings from b119 for jdk.src.share.native.sun.tracing.dtrace: JNI exception pending Message-ID: <54F88AE2.4010807@oracle.com> Please, approve this backport of a JDK 9 fix. Issue : https://bugs.openjdk.java.net/browse/JDK-8030115 Changeset : http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/607b15cdc425 Review : http://mail.openjdk.java.net/pipermail/serviceability-dev/2014-July/015259.html The patch applies cleanly to jdk8u-dev/jdk. Thanks, -JB- From rob.mckenna at oracle.com Thu Mar 5 17:09:50 2015 From: rob.mckenna at oracle.com (Rob McKenna) Date: Thu, 05 Mar 2015 17:09:50 +0000 Subject: [8u-dev] Request for approval: 8030115 - [parfait] warnings from b119 for jdk.src.share.native.sun.tracing.dtrace: JNI exception pending In-Reply-To: <54F88AE2.4010807@oracle.com> References: <54F88AE2.4010807@oracle.com> Message-ID: <54F88DDE.4040507@oracle.com> Approved. Please add an appropriate noreg keyword. -Rob On 05/03/15 16:57, Jaroslav Bachorik wrote: > Please, approve this backport of a JDK 9 fix. > > Issue : https://bugs.openjdk.java.net/browse/JDK-8030115 > Changeset : http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/607b15cdc425 > Review : > http://mail.openjdk.java.net/pipermail/serviceability-dev/2014-July/015259.html > > The patch applies cleanly to jdk8u-dev/jdk. > > Thanks, > > -JB- From attila.szegedi at oracle.com Thu Mar 5 17:34:47 2015 From: attila.szegedi at oracle.com (Attila Szegedi) Date: Thu, 5 Mar 2015 18:34:47 +0100 Subject: [8u60] Request for Approval: 8062030: Nashorn bug retrieving array property after key string concatenation Message-ID: Please approve. Bug: https://bugs.openjdk.java.net/browse/JDK-8062030 jdk9 webrev: http://cr.openjdk.java.net/~attila/8062030/webrev.00 jdk9 review thread: http://mail.openjdk.java.net/pipermail/nashorn-dev/2014-December/004023.html Changes apply cleanly to jdk8u-dev after path reshuffling from modular source code layout. Thanks, Attila. From rob.mckenna at oracle.com Thu Mar 5 18:37:53 2015 From: rob.mckenna at oracle.com (Rob McKenna) Date: Thu, 05 Mar 2015 18:37:53 +0000 Subject: [8u60] Request for Approval: 8062030: Nashorn bug retrieving array property after key string concatenation In-Reply-To: References: Message-ID: <54F8A281.7000201@oracle.com> Approved. -Rob On 05/03/15 17:34, Attila Szegedi wrote: > Please approve. > > Bug: https://bugs.openjdk.java.net/browse/JDK-8062030 > jdk9 webrev: http://cr.openjdk.java.net/~attila/8062030/webrev.00 > jdk9 review thread: http://mail.openjdk.java.net/pipermail/nashorn-dev/2014-December/004023.html > > Changes apply cleanly to jdk8u-dev after path reshuffling from modular source code layout. > > Thanks, > Attila. From anton.nashatyrev at oracle.com Fri Mar 6 13:46:12 2015 From: anton.nashatyrev at oracle.com (Anton Nashatyrev) Date: Fri, 06 Mar 2015 16:46:12 +0300 Subject: [8u-dev] Backport request: 8072900: Mouse events are captured by the wrong menu in OS X Message-ID: <54F9AFA4.1050704@oracle.com> Hello, please approve the fix backport from jdk9 to jdk8u-dev. The patch was applied strictly. Bug: https://bugs.openjdk.java.net/browse/JDK-8072900 Jdk9 Changeset: http://hg.openjdk.java.net/jdk9/client/jdk/rev/1623d30c2621 Jdk9 Review Thread: http://mail.openjdk.java.net/pipermail/swing-dev/2015-March/004233.html Thanks! Anton. From magnus.ihse.bursie at oracle.com Fri Mar 6 14:09:20 2015 From: magnus.ihse.bursie at oracle.com (Magnus Ihse Bursie) Date: Fri, 06 Mar 2015 15:09:20 +0100 Subject: RFR: JDK-8074554 Create custom hook for running after AC_OUTPUT (8u backport) Message-ID: <54F9B510.4040509@oracle.com> Please review this backport of JDK-8074554 to 8u. While it did not automatically apply cleanly, the difference was trivially resolved. diff --git a/common/autoconf/configure.ac b/common/autoconf/configure.ac --- a/common/autoconf/configure.ac +++ b/common/autoconf/configure.ac @@ -53,6 +53,7 @@ AC_DEFUN_ONCE([CUSTOM_EARLY_HOOK]) AC_DEFUN_ONCE([CUSTOM_LATE_HOOK]) +AC_DEFUN_ONCE([CUSTOM_CONFIG_OUTPUT_GENERATED_HOOK]) # This line needs to be here, verbatim, after all includes and the dummy hook # definitions. It is replaced with custom functionality when building @@ -237,6 +238,7 @@ # Create the actual output files. Now the main work of configure is done. AC_OUTPUT +CUSTOM_CONFIG_OUTPUT_GENERATED_HOOK # Try to move the config.log file to the output directory. if test -e ./config.log; then /Magnus From erik.joelsson at oracle.com Fri Mar 6 14:27:54 2015 From: erik.joelsson at oracle.com (Erik Joelsson) Date: Fri, 06 Mar 2015 15:27:54 +0100 Subject: RFR: JDK-8074554 Create custom hook for running after AC_OUTPUT (8u backport) In-Reply-To: <54F9B510.4040509@oracle.com> References: <54F9B510.4040509@oracle.com> Message-ID: <54F9B96A.8040004@oracle.com> Looks good. /Erik On 2015-03-06 15:09, Magnus Ihse Bursie wrote: > Please review this backport of JDK-8074554 to 8u. While it did not > automatically apply cleanly, the difference was trivially resolved. > > diff --git a/common/autoconf/configure.ac b/common/autoconf/configure.ac > --- a/common/autoconf/configure.ac > +++ b/common/autoconf/configure.ac > @@ -53,6 +53,7 @@ > > AC_DEFUN_ONCE([CUSTOM_EARLY_HOOK]) > AC_DEFUN_ONCE([CUSTOM_LATE_HOOK]) > +AC_DEFUN_ONCE([CUSTOM_CONFIG_OUTPUT_GENERATED_HOOK]) > > # This line needs to be here, verbatim, after all includes and the > dummy hook > # definitions. It is replaced with custom functionality when building > @@ -237,6 +238,7 @@ > > # Create the actual output files. Now the main work of configure is > done. > AC_OUTPUT > +CUSTOM_CONFIG_OUTPUT_GENERATED_HOOK > > # Try to move the config.log file to the output directory. > if test -e ./config.log; then > > /Magnus From magnus.ihse.bursie at oracle.com Fri Mar 6 14:41:37 2015 From: magnus.ihse.bursie at oracle.com (Magnus Ihse Bursie) Date: Fri, 06 Mar 2015 15:41:37 +0100 Subject: [8u-dev] Request for approval for JDK-8074554 Create custom hook for running after AC_OUTPUT In-Reply-To: <54F9B96A.8040004@oracle.com> References: <54F9B510.4040509@oracle.com> <54F9B96A.8040004@oracle.com> Message-ID: <54F9BCA1.2020204@oracle.com> Requesting approval to backport this JDK9 fix to jdk8u-dev. Bug: https://bugs.openjdk.java.net/browse/JDK-8074554 Original JDK9 changeset: http://hg.openjdk.java.net/jdk9/client/rev/8b9bd5ba445e /Magnus On 2015-03-06 15:27, Erik Joelsson wrote: > Looks good. > > /Erik > > On 2015-03-06 15:09, Magnus Ihse Bursie wrote: >> Please review this backport of JDK-8074554 to 8u. While it did not >> automatically apply cleanly, the difference was trivially resolved. >> >> diff --git a/common/autoconf/configure.ac b/common/autoconf/configure.ac >> --- a/common/autoconf/configure.ac >> +++ b/common/autoconf/configure.ac >> @@ -53,6 +53,7 @@ >> >> AC_DEFUN_ONCE([CUSTOM_EARLY_HOOK]) >> AC_DEFUN_ONCE([CUSTOM_LATE_HOOK]) >> +AC_DEFUN_ONCE([CUSTOM_CONFIG_OUTPUT_GENERATED_HOOK]) >> >> # This line needs to be here, verbatim, after all includes and the >> dummy hook >> # definitions. It is replaced with custom functionality when building >> @@ -237,6 +238,7 @@ >> >> # Create the actual output files. Now the main work of configure is >> done. >> AC_OUTPUT >> +CUSTOM_CONFIG_OUTPUT_GENERATED_HOOK >> >> # Try to move the config.log file to the output directory. >> if test -e ./config.log; then >> >> /Magnus > From gitne at gmx.de Fri Mar 6 14:49:47 2015 From: gitne at gmx.de (Jacob Wisor) Date: Fri, 06 Mar 2015 15:49:47 +0100 Subject: RFR: Let jar tool print timestamps in default locale date and time in format Message-ID: <54F9BE8B.8040308@gmx.de> Hello there! I have stumbled upon the jar tool always printing nominally the same date and time for a JarEntry, regardless of the timezone set. Confused by this, I dug into the code only to find out that the zip file format does neither store timezone information nor timestamps in UTC. This basically means that timestamps in jar/zip files are or can be assumed to always be in the local timezone, hence there is no conceivable fix for this problem, except for altering the zip file format specification. However, timestamps are printed in the en_US locale currently. This locale is /almost/ as arbitrary as every other. So, this patch makes the jar tool print timestamps in the default locale date and time format. If anybody is bothered by this change they can still explicitly set the user.language property or the system's locale to en_US. If anybody /really/ wants to parse jar tool's timestamps output they should probably be relying on ISO formatted timestamps. I can add this functionality as a flag. Anyways, please review this patch. Regards, Jacob -------------- next part -------------- A non-text attachment was scrubbed... Name: Print JAR file entry timestamps in default locale date and time format.patch Type: text/x-patch Size: 1059 bytes Desc: not available URL: From rob.mckenna at oracle.com Fri Mar 6 17:31:01 2015 From: rob.mckenna at oracle.com (Rob McKenna) Date: Fri, 06 Mar 2015 17:31:01 +0000 Subject: [8u-dev] Backport request: 8072900: Mouse events are captured by the wrong menu in OS X In-Reply-To: <54F9AFA4.1050704@oracle.com> References: <54F9AFA4.1050704@oracle.com> Message-ID: <54F9E455.1040601@oracle.com> Approved. -Rob On 06/03/15 13:46, Anton Nashatyrev wrote: > Hello, > > please approve the fix backport from jdk9 to jdk8u-dev. The patch > was applied strictly. > > Bug: https://bugs.openjdk.java.net/browse/JDK-8072900 > Jdk9 Changeset: > http://hg.openjdk.java.net/jdk9/client/jdk/rev/1623d30c2621 > Jdk9 Review Thread: > http://mail.openjdk.java.net/pipermail/swing-dev/2015-March/004233.html > > Thanks! > Anton. From rob.mckenna at oracle.com Fri Mar 6 17:31:47 2015 From: rob.mckenna at oracle.com (Rob McKenna) Date: Fri, 06 Mar 2015 17:31:47 +0000 Subject: [8u-dev] Request for approval for JDK-8074554 Create custom hook for running after AC_OUTPUT In-Reply-To: <54F9BCA1.2020204@oracle.com> References: <54F9B510.4040509@oracle.com> <54F9B96A.8040004@oracle.com> <54F9BCA1.2020204@oracle.com> Message-ID: <54F9E483.3000208@oracle.com> Approved. Please add a suitable noreg keyword. -Rob On 06/03/15 14:41, Magnus Ihse Bursie wrote: > Requesting approval to backport this JDK9 fix to jdk8u-dev. > > Bug: https://bugs.openjdk.java.net/browse/JDK-8074554 > Original JDK9 changeset: > http://hg.openjdk.java.net/jdk9/client/rev/8b9bd5ba445e > > /Magnus > > On 2015-03-06 15:27, Erik Joelsson wrote: >> Looks good. >> >> /Erik >> >> On 2015-03-06 15:09, Magnus Ihse Bursie wrote: >>> Please review this backport of JDK-8074554 to 8u. While it did not >>> automatically apply cleanly, the difference was trivially resolved. >>> >>> diff --git a/common/autoconf/configure.ac >>> b/common/autoconf/configure.ac >>> --- a/common/autoconf/configure.ac >>> +++ b/common/autoconf/configure.ac >>> @@ -53,6 +53,7 @@ >>> >>> AC_DEFUN_ONCE([CUSTOM_EARLY_HOOK]) >>> AC_DEFUN_ONCE([CUSTOM_LATE_HOOK]) >>> +AC_DEFUN_ONCE([CUSTOM_CONFIG_OUTPUT_GENERATED_HOOK]) >>> >>> # This line needs to be here, verbatim, after all includes and the >>> dummy hook >>> # definitions. It is replaced with custom functionality when building >>> @@ -237,6 +238,7 @@ >>> >>> # Create the actual output files. Now the main work of configure is >>> done. >>> AC_OUTPUT >>> +CUSTOM_CONFIG_OUTPUT_GENERATED_HOOK >>> >>> # Try to move the config.log file to the output directory. >>> if test -e ./config.log; then >>> >>> /Magnus >> > From alejandro.murillo at oracle.com Fri Mar 6 17:55:37 2015 From: alejandro.murillo at oracle.com (Alejandro E Murillo) Date: Fri, 06 Mar 2015 10:55:37 -0700 Subject: [8u60] Request for approval for bulk integration of hs25.60-b06 Message-ID: <54F9EA19.3060307@oracle.com> Requesting approval to integrate hs25.60-b06 into jdk8u60-b06. A webrev is available at: http://cr.openjdk.java.net/~amurillo/8u60/hs25.60-b06-jdk8u60-b06.webrev/ Pre-integration testing is in progress; the integration will proceed only after SQE has analyzed the results and approved. The fixes in the proposed integration are below. All have undergone nightly testing and are already in a jdk9 repository. 7176220: 'Full GC' events miss date stamp information occasionally 8025667: Warning from b62 for hotspot.agent.src.os.solaris.proc: use after free 8049049: Unportable format string argument mismatch in hotspot/agent/src/os/solaris/proc/saproc.cpp 8073944: Simplify ArgumentsExt and remove unneeded functionallity 8074010: followup to 8072383 8074038: new hotspot build - hs25.60-b06 -- Alejandro From rob.mckenna at oracle.com Fri Mar 6 18:18:31 2015 From: rob.mckenna at oracle.com (Rob McKenna) Date: Fri, 06 Mar 2015 18:18:31 +0000 Subject: [8u60] Request for approval for bulk integration of hs25.60-b06 In-Reply-To: <54F9EA19.3060307@oracle.com> References: <54F9EA19.3060307@oracle.com> Message-ID: <54F9EF77.4050104@oracle.com> Approved. -Rob On 06/03/15 17:55, Alejandro E Murillo wrote: > Requesting approval to integrate hs25.60-b06 into jdk8u60-b06. > > A webrev is available at: > > http://cr.openjdk.java.net/~amurillo/8u60/hs25.60-b06-jdk8u60-b06.webrev/ > > Pre-integration testing is in progress; the integration will proceed > only after SQE has analyzed the results and approved. > > The fixes in the proposed integration are below. All have undergone > nightly testing and are already in a jdk9 repository. > > 7176220: 'Full GC' events miss date stamp information occasionally > 8025667: Warning from b62 for hotspot.agent.src.os.solaris.proc: use > after free > 8049049: Unportable format string argument mismatch in > hotspot/agent/src/os/solaris/proc/saproc.cpp > 8073944: Simplify ArgumentsExt and remove unneeded functionallity > 8074010: followup to 8072383 > 8074038: new hotspot build - hs25.60-b06 > From dean.long at oracle.com Fri Mar 6 20:21:20 2015 From: dean.long at oracle.com (Dean Long) Date: Fri, 06 Mar 2015 12:21:20 -0800 Subject: [8u60] Request for backport approval: 8064357: AARCH64: Top-level JDK changes Message-ID: <54FA0C40.9070807@oracle.com> I would like to backport "8064357: AARCH64: Top-level JDK changes" to 8u60. Bug: https://bugs.openjdk.java.net/browse/JDK-8064357 Original review thread: http://mail.openjdk.java.net/pipermail/build-dev/2014-November/013587.html The jdk9 changeset is here: http://hg.openjdk.java.net/jdk9/dev/rev/0282b3f5f749 The patch imported cleanly. Let me know if there are any concerns about this backport. thanks, dl From rob.mckenna at oracle.com Fri Mar 6 22:01:24 2015 From: rob.mckenna at oracle.com (Rob McKenna) Date: Fri, 06 Mar 2015 22:01:24 +0000 Subject: [8u60] Request for backport approval: 8064357: AARCH64: Top-level JDK changes In-Reply-To: <54FA0C40.9070807@oracle.com> References: <54FA0C40.9070807@oracle.com> Message-ID: <54FA23B4.7040106@oracle.com> Approved. Please add an appropriate noreg keyword to the bug. -Rob On 06/03/15 20:21, Dean Long wrote: > I would like to backport "8064357: AARCH64: Top-level JDK changes" to > 8u60. > > Bug: > > https://bugs.openjdk.java.net/browse/JDK-8064357 > > Original review thread: > > http://mail.openjdk.java.net/pipermail/build-dev/2014-November/013587.html > > > The jdk9 changeset is here: > > http://hg.openjdk.java.net/jdk9/dev/rev/0282b3f5f749 > > The patch imported cleanly. Let me know if there are any concerns about > this backport. > > thanks, > > dl From dean.long at oracle.com Sat Mar 7 23:29:40 2015 From: dean.long at oracle.com (Dean Long) Date: Sat, 07 Mar 2015 15:29:40 -0800 Subject: [8u60] Request for backport approval: 8064357: AARCH64: Top-level JDK changes In-Reply-To: <54FA23B4.7040106@oracle.com> References: <54FA0C40.9070807@oracle.com> <54FA23B4.7040106@oracle.com> Message-ID: <54FB89E4.7090705@oracle.com> Will do. Thanks. dl On 3/6/2015 2:01 PM, Rob McKenna wrote: > Approved. Please add an appropriate noreg keyword to the bug. > > -Rob > > On 06/03/15 20:21, Dean Long wrote: >> I would like to backport "8064357: AARCH64: Top-level JDK changes" to >> 8u60. >> >> Bug: >> >> https://bugs.openjdk.java.net/browse/JDK-8064357 >> >> Original review thread: >> >> http://mail.openjdk.java.net/pipermail/build-dev/2014-November/013587.html >> >> >> The jdk9 changeset is here: >> >> http://hg.openjdk.java.net/jdk9/dev/rev/0282b3f5f749 >> >> The patch imported cleanly. Let me know if there are any concerns about >> this backport. >> >> thanks, >> >> dl > From gitne at gmx.de Sun Mar 8 10:17:31 2015 From: gitne at gmx.de (Jacob Wisor) Date: Sun, 08 Mar 2015 11:17:31 +0100 Subject: RFR [rfc]: Add pl localization to jar tool Message-ID: <54FC21BB.4050902@gmx.de> Hello there! I have added pl localization to the JAR tool. More tools are coming! ;-) Regards, Jacob -------------- next part -------------- A non-text attachment was scrubbed... Name: Add pl localization of JAR tool.patch Type: text/x-patch Size: 4553 bytes Desc: not available URL: From gitne at gmx.de Sun Mar 8 11:08:32 2015 From: gitne at gmx.de (Jacob Wisor) Date: Sun, 08 Mar 2015 12:08:32 +0100 Subject: RFR: Add de and pl localizations to native2ascii tool Message-ID: <54FC2DB0.3090407@gmx.de> Hello there! I have added de and pl localizations to the native2ascii tool. And, I have migrated the other existing localizations to property file resource bundles, just like in the other tools. Regards, Jacob -------------- next part -------------- A non-text attachment was scrubbed... Name: Add de and pl localizations to native2ascii tool.patch Type: text/x-patch Size: 15486 bytes Desc: not available URL: From staffan.larsen at oracle.com Mon Mar 9 09:28:58 2015 From: staffan.larsen at oracle.com (Staffan Larsen) Date: Mon, 9 Mar 2015 10:28:58 +0100 Subject: [8u60] Request for approval for CR 8025636: Hide lambda proxy frames in stacktraces Message-ID: Please approve the following backport. bug: https://bugs.openjdk.java.net/browse/JDK-8025636 jdk 9 changeset: http://hg.openjdk.java.net/jdk9/dev/jdk/rev/88a6f5874f09 review thread: http://mail.openjdk.java.net/pipermail/core-libs-dev/2015-February/031216.html Thanks, /Staffan From sean.coffey at oracle.com Mon Mar 9 09:42:05 2015 From: sean.coffey at oracle.com (=?windows-1252?Q?Se=E1n_Coffey?=) Date: Mon, 09 Mar 2015 09:42:05 +0000 Subject: [8u60] Request for approval for CR 8025636: Hide lambda proxy frames in stacktraces In-Reply-To: References: Message-ID: <54FD6AED.4070703@oracle.com> Approved. regards, Sean. On 09/03/2015 09:28, Staffan Larsen wrote: > Please approve the following backport. > > bug: https://bugs.openjdk.java.net/browse/JDK-8025636 > jdk 9 changeset: http://hg.openjdk.java.net/jdk9/dev/jdk/rev/88a6f5874f09 > review thread: http://mail.openjdk.java.net/pipermail/core-libs-dev/2015-February/031216.html > > Thanks, > /Staffan From hannes.wallnoefer at oracle.com Mon Mar 9 12:26:32 2015 From: hannes.wallnoefer at oracle.com (Hannes Wallnoefer) Date: Mon, 09 Mar 2015 13:26:32 +0100 Subject: [8u60] Request for approval for JDK-8074545 and JDK-8074556 Message-ID: <54FD9178.6090301@oracle.com> Please approve backport of two Nashorn bugs to 8u60: Bug: https://bugs.openjdk.java.net/browse/JDK-8074545 Webrev: http://cr.openjdk.java.net/~hannesw/8074545/ Review thread: http://mail.openjdk.java.net/pipermail/nashorn-dev/2015-March/004265.html Bug: https://bugs.openjdk.java.net/browse/JDK-8074556 Webrev: http://cr.openjdk.java.net/~hannesw/8074556/ Review thread: http://mail.openjdk.java.net/pipermail/nashorn-dev/2015-March/004267.html The changes apply cleanly to 8u-dev after reshuffling path names. Thanks, Hannes From sean.coffey at oracle.com Mon Mar 9 13:14:40 2015 From: sean.coffey at oracle.com (=?UTF-8?B?U2XDoW4gQ29mZmV5?=) Date: Mon, 09 Mar 2015 13:14:40 +0000 Subject: [8u60] Request for approval for JDK-8074545 and JDK-8074556 In-Reply-To: <54FD9178.6090301@oracle.com> References: <54FD9178.6090301@oracle.com> Message-ID: <54FD9CC0.6050502@oracle.com> Approved. regards, Sean. On 09/03/15 12:26, Hannes Wallnoefer wrote: > Please approve backport of two Nashorn bugs to 8u60: > > Bug: https://bugs.openjdk.java.net/browse/JDK-8074545 > Webrev: http://cr.openjdk.java.net/~hannesw/8074545/ > Review thread: > http://mail.openjdk.java.net/pipermail/nashorn-dev/2015-March/004265.html > > Bug: https://bugs.openjdk.java.net/browse/JDK-8074556 > Webrev: http://cr.openjdk.java.net/~hannesw/8074556/ > Review thread: > http://mail.openjdk.java.net/pipermail/nashorn-dev/2015-March/004267.html > > The changes apply cleanly to 8u-dev after reshuffling path names. > > Thanks, > Hannes From claes.redestad at oracle.com Mon Mar 9 13:22:22 2015 From: claes.redestad at oracle.com (Claes Redestad) Date: Mon, 09 Mar 2015 14:22:22 +0100 Subject: [8u60] Request for approval for JDK-8073497: Lazy conversion of ZipEntry time Message-ID: <54FD9E8E.4000201@oracle.com> Hi, please approve backporting this to 8u60. Patch applies cleanly after unshuffling Bug: https://bugs.openjdk.java.net/browse/JDK-8073497 Webrev: http://cr.openjdk.java.net/~redestad/jdk9/8073497/webrev.6/ Review thread: http://mail.openjdk.java.net/pipermail/core-libs-dev/2015-February/031703.html /Claes From sean.coffey at oracle.com Mon Mar 9 13:52:26 2015 From: sean.coffey at oracle.com (=?UTF-8?B?U2XDoW4gQ29mZmV5?=) Date: Mon, 09 Mar 2015 13:52:26 +0000 Subject: [8u60] Request for approval for JDK-8073497: Lazy conversion of ZipEntry time In-Reply-To: <54FD9E8E.4000201@oracle.com> References: <54FD9E8E.4000201@oracle.com> Message-ID: <54FDA59A.6090506@oracle.com> Approved. regards, Sean. On 09/03/15 13:22, Claes Redestad wrote: > Hi, > > please approve backporting this to 8u60. Patch applies cleanly after > unshuffling > > Bug: https://bugs.openjdk.java.net/browse/JDK-8073497 > Webrev: http://cr.openjdk.java.net/~redestad/jdk9/8073497/webrev.6/ > Review thread: > http://mail.openjdk.java.net/pipermail/core-libs-dev/2015-February/031703.html > > /Claes From lana.steuck at oracle.com Mon Mar 9 23:16:52 2015 From: lana.steuck at oracle.com (lana.steuck at oracle.com) Date: Mon, 9 Mar 2015 16:16:52 -0700 (PDT) Subject: jdk8u-b06: jdk8u-dev Message-ID: <201503092316.t29NGqn3011054@sc11152554.us.oracle.com> http://hg.openjdk.java.net/jdk8u/jdk8u/rev/74dbdbcb6aac http://hg.openjdk.java.net/jdk8u/jdk8u/nashorn/rev/4b7613f08fd3 http://hg.openjdk.java.net/jdk8u/jdk8u/langtools/rev/44d168f9ad16 http://hg.openjdk.java.net/jdk8u/jdk8u/jdk/rev/3a8ecea921f6 http://hg.openjdk.java.net/jdk8u/jdk8u/jaxws/rev/a22a9460d53f http://hg.openjdk.java.net/jdk8u/jdk8u/jaxp/rev/b265b3944437 http://hg.openjdk.java.net/jdk8u/jdk8u/hotspot/rev/db433ae5c123 http://hg.openjdk.java.net/jdk8u/jdk8u/corba/rev/587b01196646 --- All the fixes will be tested during promotion (no PIT testing at this point): List of all fixes: =================== JDK-8068040 client-libs [macosx] Combo box consuming ENTER key events JDK-8072676 client-libs [macosx] Jtree icon painted over label when scrollbars present in wind JDK-8071705 client-libs Java application menu misbehaves when running multiple screen stacked JDK-8072069 client-libs Toolkit.getScreenInsets() doesn't update if insets change JDK-8067748 core-libs (process) Child is terminated when parent's console is closed [win] JDK-8067846 core-libs (sctp) InternalError when receiving SendFailedNotification JDK-8071599 core-libs (so) Socket adapter sendUrgentData throws IllegalBlockingMode when cha JDK-8073354 core-libs [TEST_BUG] TimSortStackSize2.java: test cleanup: make test run with si JDK-8074125 core-libs Add SerializedLogRecord test to jdk8u JDK-8072596 core-libs Arrays.asList results in ClassCastException with a JS array JDK-8068889 core-libs Calling a @FunctionalInterface from JS leaks internal objects JDK-8073707 core-libs const re-assignment should not reported as a "early error" JDK-8068994 core-libs Forgot to add a test model to JDK-8068573 JDK-8071447 core-libs IBM1166 Locale Request for Kazakh characters JDK-8074021 core-libs Indirect eval fails when used as an element of an array or as a proper JDK-8066842 core-libs java.math.BigDecimal.divide(BigDecimal, RoundingMode) produces incorre JDK-8072000 core-libs New compiler warning after JDK-8067139 JDK-8068573 core-libs POJO setter using [] syntax throws an exception JDK-8069002 core-libs REGRESSION: test/script/external/test262/test/suite/ch11/11.2/11.2.3/S JDK-7178362 core-libs Socket impls should ignore unsupported proxy types rather than throwin JDK-8072909 core-libs TimSort fails with ArrayIndexOutOfBoundsException on worst case long a JDK-8073124 core-libs Tune test and document TimSort runs stack size increase JDK-8072602 core-libs Unpredictable timezone on Windows when OS's timezone is not found in t JDK-8068790 core-libs ZipEntry/JarEntry.setCreation/LastAccessTime(null) don't throw NPE as JDK-8064331 core-svc JavaSecurityAccess.doIntersectionPrivilege() drops the information abo JDK-8068969 deploy Add missing information to AppModel JDK-8058688 docs RSA links broken in the Java Cryptography Architecture Standard Algori JDK-8069122 globalization l10n resource file update for JDK-8068491 JDK-8069590 hotspot AIX port of "8050807: Better performing performance data handling" JDK-8069412 hotspot Locks need better debug-printing support JDK-8072932 hotspot Test fails with java.security.AccessControlException: access denied (" JDK-8067330 infrastructure ZERO_ARCHDEF incorrectly defined for PPC/PPC64 architectures JDK-8041740 security-libs [TEST_BUG] Test sun/security/tools/keytool/ListKeychainStore.sh fails JDK-8069072 security-libs Improve GHASH performance JDK-8039921 security-libs SHA1WithDSA with key > 1024 bits not working JDK-8022313 security-libs sun/security/pkcs11/rsa/TestKeyPairGenerator.java failed in aurora JDK-8068517 tools Compiler may generate wrong InnerClasses attribute for static enum ref JDK-8073972 tools Deprecate Multi-Version Java Launcher (mJRE) for JDK8 From david.holmes at oracle.com Tue Mar 10 03:40:55 2015 From: david.holmes at oracle.com (David Holmes) Date: Tue, 10 Mar 2015 13:40:55 +1000 Subject: Fix for 8030115 breaks windows build Message-ID: <54FE67C7.4050009@oracle.com> The fix for 8030115 breaks the build on windows: c:/jprt/T/P1/021115.drlong/s/jdk/src/share/native/sun/tracing/dtrace/JVM.c(200) : error C2143: syntax error : missing ';' before 'type' c:/jprt/T/P1/021115.drlong/s/jdk/src/share/native/sun/tracing/dtrace/JVM.c(201) : error C2065: 'count' : undeclared identifier c:/jprt/T/P1/021115.drlong/s/jdk/src/share/native/sun/tracing/dtrace/JVM.c(201) : error C2065: 'count' : undeclared identifier c:/jprt/T/P1/021115.drlong/s/jdk/src/share/native/sun/tracing/dtrace/JVM.c(202) : error C2065: 'count' : undeclared identifier c:/jprt/T/P1/021115.drlong/s/jdk/src/share/native/sun/tracing/dtrace/JVM.c(204) : error C2065: 'count' : undeclared identifier c:/jprt/T/P1/021115.drlong/s/jdk/src/share/native/sun/tracing/dtrace/JVM.c(212) : error C2065: 'count' : undeclared identifier make[2]: *** [/cygdrive/c/jprt/T/P1/021115.drlong/s/build/windows-x86_64-normal-server-fastdebug/jdk/objs/libjsdt/JVM.obj] Error 2 The older MSC compiler doesn't support declarations in the middle of blocks IIRC so barfs on this: @@ -195,16 +197,24 @@ jvm_providers = (JVM_DTraceProvider*)calloc( num_providers, sizeof(*jvm_providers)); - for (i = 0; i < num_providers; ++i) { - JVM_DTraceProvider* p = &(jvm_providers[i]); + int count = 0; + for (; count < num_providers; ++count) { Filed: https://bugs.openjdk.java.net/browse/JDK-8074795 Not sure how this has escaped notice till now but the nightly build has been failing on Windows since this was pushed! :( I'm testing the simple fix and will send out the RFR/A as soon as it is confirmed. David From david.holmes at oracle.com Tue Mar 10 03:51:26 2015 From: david.holmes at oracle.com (David Holmes) Date: Tue, 10 Mar 2015 13:51:26 +1000 Subject: Fix for 8030115 breaks windows build In-Reply-To: <54FE67C7.4050009@oracle.com> References: <54FE67C7.4050009@oracle.com> Message-ID: <54FE6A3E.4000706@oracle.com> Talk about history repeating itself, lessons not learned etc. :( The original fix was broken when pushed to 9 and subsequently fixed again under: https://bugs.openjdk.java.net/browse/JDK-8053902 Hence we just need a backport of JDK-8053902. I'll send the approval request immediately. David On 10/03/2015 1:40 PM, David Holmes wrote: > The fix for 8030115 breaks the build on windows: > > c:/jprt/T/P1/021115.drlong/s/jdk/src/share/native/sun/tracing/dtrace/JVM.c(200) > : error C2143: syntax error : missing ';' before 'type' > c:/jprt/T/P1/021115.drlong/s/jdk/src/share/native/sun/tracing/dtrace/JVM.c(201) > : error C2065: 'count' : undeclared identifier > c:/jprt/T/P1/021115.drlong/s/jdk/src/share/native/sun/tracing/dtrace/JVM.c(201) > : error C2065: 'count' : undeclared identifier > c:/jprt/T/P1/021115.drlong/s/jdk/src/share/native/sun/tracing/dtrace/JVM.c(202) > : error C2065: 'count' : undeclared identifier > c:/jprt/T/P1/021115.drlong/s/jdk/src/share/native/sun/tracing/dtrace/JVM.c(204) > : error C2065: 'count' : undeclared identifier > c:/jprt/T/P1/021115.drlong/s/jdk/src/share/native/sun/tracing/dtrace/JVM.c(212) > : error C2065: 'count' : undeclared identifier > make[2]: *** > [/cygdrive/c/jprt/T/P1/021115.drlong/s/build/windows-x86_64-normal-server-fastdebug/jdk/objs/libjsdt/JVM.obj] > Error 2 > > The older MSC compiler doesn't support declarations in the middle of > blocks IIRC so barfs on this: > > @@ -195,16 +197,24 @@ > jvm_providers = (JVM_DTraceProvider*)calloc( > num_providers, sizeof(*jvm_providers)); > > - for (i = 0; i < num_providers; ++i) { > - JVM_DTraceProvider* p = &(jvm_providers[i]); > + int count = 0; > + for (; count < num_providers; ++count) { > > > Filed: > > https://bugs.openjdk.java.net/browse/JDK-8074795 > > Not sure how this has escaped notice till now but the nightly build has > been failing on Windows since this was pushed! :( > > I'm testing the simple fix and will send out the RFR/A as soon as it is > confirmed. > > David > From david.holmes at oracle.com Tue Mar 10 04:03:54 2015 From: david.holmes at oracle.com (David Holmes) Date: Tue, 10 Mar 2015 14:03:54 +1000 Subject: URGENT [8u60] Backport: 8053902: Fix for 8030115 breaks build on Windows and Solaris Message-ID: <54FE6D2A.2050409@oracle.com> Please approve the direct backport of this fix to JDK 8u-dev as it is causing build failures. The bug: https://bugs.openjdk.java.net/browse/JDK-8053902 The review thread ... well I couldn't actually find a review thread as such, only this notification: http://mail.openjdk.java.net/pipermail/serviceability-dev/2014-July/015286.html The JDK 9 changeset: http://hg.openjdk.java.net/jdk9/dev/jdk/rev/2b1a17af5308 Thanks, David From vladimir.kozlov at oracle.com Tue Mar 10 05:21:23 2015 From: vladimir.kozlov at oracle.com (Vladimir Kozlov) Date: Mon, 09 Mar 2015 22:21:23 -0700 Subject: URGENT [8u60] Backport: 8053902: Fix for 8030115 breaks build on Windows and Solaris In-Reply-To: <54FE6D2A.2050409@oracle.com> References: <54FE6D2A.2050409@oracle.com> Message-ID: <54FE7F53.3010700@oracle.com> Looks good. Thanks, Vladimir On 3/9/15 9:03 PM, David Holmes wrote: > Please approve the direct backport of this fix to JDK 8u-dev as it is causing build failures. > > The bug: https://bugs.openjdk.java.net/browse/JDK-8053902 > > The review thread ... well I couldn't actually find a review thread as such, only this notification: > > http://mail.openjdk.java.net/pipermail/serviceability-dev/2014-July/015286.html > > The JDK 9 changeset: > > http://hg.openjdk.java.net/jdk9/dev/jdk/rev/2b1a17af5308 > > Thanks, > David From david.holmes at oracle.com Tue Mar 10 05:27:14 2015 From: david.holmes at oracle.com (David Holmes) Date: Tue, 10 Mar 2015 15:27:14 +1000 Subject: URGENT [8u60] Backport: 8053902: Fix for 8030115 breaks build on Windows and Solaris In-Reply-To: <54FE7F53.3010700@oracle.com> References: <54FE6D2A.2050409@oracle.com> <54FE7F53.3010700@oracle.com> Message-ID: <54FE80B2.9010007@oracle.com> On 10/03/2015 3:21 PM, Vladimir Kozlov wrote: > Looks good. Thanks Vladimir, but this needs approval from the 8u maintainer(s). Though you've reminded me that we may want to pull this into jdk8u/hs-dev as well. David > Thanks, > Vladimir > > On 3/9/15 9:03 PM, David Holmes wrote: >> Please approve the direct backport of this fix to JDK 8u-dev as it is >> causing build failures. >> >> The bug: https://bugs.openjdk.java.net/browse/JDK-8053902 >> >> The review thread ... well I couldn't actually find a review thread as >> such, only this notification: >> >> http://mail.openjdk.java.net/pipermail/serviceability-dev/2014-July/015286.html >> >> >> The JDK 9 changeset: >> >> http://hg.openjdk.java.net/jdk9/dev/jdk/rev/2b1a17af5308 >> >> Thanks, >> David From sean.coffey at oracle.com Tue Mar 10 09:10:19 2015 From: sean.coffey at oracle.com (=?UTF-8?B?U2XDoW4gQ29mZmV5?=) Date: Tue, 10 Mar 2015 09:10:19 +0000 Subject: URGENT [8u60] Backport: 8053902: Fix for 8030115 breaks build on Windows and Solaris In-Reply-To: <54FE80B2.9010007@oracle.com> References: <54FE6D2A.2050409@oracle.com> <54FE7F53.3010700@oracle.com> <54FE80B2.9010007@oracle.com> Message-ID: <54FEB4FB.7070704@oracle.com> Approved! Please add noreg-build label to bug report. regards, Sean. On 10/03/2015 05:27, David Holmes wrote: > On 10/03/2015 3:21 PM, Vladimir Kozlov wrote: >> Looks good. > > Thanks Vladimir, but this needs approval from the 8u maintainer(s). > > Though you've reminded me that we may want to pull this into > jdk8u/hs-dev as well. > > David > >> Thanks, >> Vladimir >> >> On 3/9/15 9:03 PM, David Holmes wrote: >>> Please approve the direct backport of this fix to JDK 8u-dev as it is >>> causing build failures. >>> >>> The bug: https://bugs.openjdk.java.net/browse/JDK-8053902 >>> >>> The review thread ... well I couldn't actually find a review thread as >>> such, only this notification: >>> >>> http://mail.openjdk.java.net/pipermail/serviceability-dev/2014-July/015286.html >>> >>> >>> >>> The JDK 9 changeset: >>> >>> http://hg.openjdk.java.net/jdk9/dev/jdk/rev/2b1a17af5308 >>> >>> Thanks, >>> David From david.holmes at oracle.com Tue Mar 10 10:53:32 2015 From: david.holmes at oracle.com (David Holmes) Date: Tue, 10 Mar 2015 20:53:32 +1000 Subject: URGENT [8u60] Backport: 8053902: Fix for 8030115 breaks build on Windows and Solaris In-Reply-To: <54FEB4FB.7070704@oracle.com> References: <54FE6D2A.2050409@oracle.com> <54FE7F53.3010700@oracle.com> <54FE80B2.9010007@oracle.com> <54FEB4FB.7070704@oracle.com> Message-ID: <54FECD2C.5050101@oracle.com> On 10/03/2015 7:10 PM, Se?n Coffey wrote: > Approved! Please add noreg-build label to bug report. Thanks Sean - done. David > regards, > Sean. > > On 10/03/2015 05:27, David Holmes wrote: >> On 10/03/2015 3:21 PM, Vladimir Kozlov wrote: >>> Looks good. >> >> Thanks Vladimir, but this needs approval from the 8u maintainer(s). >> >> Though you've reminded me that we may want to pull this into >> jdk8u/hs-dev as well. >> >> David >> >>> Thanks, >>> Vladimir >>> >>> On 3/9/15 9:03 PM, David Holmes wrote: >>>> Please approve the direct backport of this fix to JDK 8u-dev as it is >>>> causing build failures. >>>> >>>> The bug: https://bugs.openjdk.java.net/browse/JDK-8053902 >>>> >>>> The review thread ... well I couldn't actually find a review thread as >>>> such, only this notification: >>>> >>>> http://mail.openjdk.java.net/pipermail/serviceability-dev/2014-July/015286.html >>>> >>>> >>>> >>>> The JDK 9 changeset: >>>> >>>> http://hg.openjdk.java.net/jdk9/dev/jdk/rev/2b1a17af5308 >>>> >>>> Thanks, >>>> David > From nakul.natu at oracle.com Tue Mar 10 19:04:56 2015 From: nakul.natu at oracle.com (Nakul Natu) Date: Tue, 10 Mar 2015 12:04:56 -0700 Subject: [8u-dev] Request for Approval to backport JDK-8066436: Minimize can cause window to disappear on osx Message-ID: <54FF4058.3030200@oracle.com> Hello, Please approve backporting this to 8u-dev. Patch applies cleanly after paths are corrected for jdk8 Bug: https://bugs.openjdk.java.net/browse/JDK-8066436 Webrev: /http://cr.openjdk.java.net/~van/8066436/webrev.01/ / Review thread: http://mail.openjdk.java.net/pipermail/awt-dev/2015-March/009018.html Regards, Nakul From dean.long at oracle.com Tue Mar 10 20:28:59 2015 From: dean.long at oracle.com (Dean Long) Date: Tue, 10 Mar 2015 13:28:59 -0700 Subject: [8u60] Request for backport approval: 8069057 and 8072732 Message-ID: <54FF540B.8010601@oracle.com> I would like to backport "8069057: Make sure configure is run by bash" and the followup fix "8072732: Regression in configure due to JDK-8069057" to 8u60. Both are needed by the previously approved backport of "8064357: AARCH64: Top-level JDK changes". Without it, JPRT fails when trying to use /bin/sh instead of /bin/bash. Bugs: https://bugs.openjdk.java.net/browse/JDK-8069057 https://bugs.openjdk.java.net/browse/JDK-8072732 Original review threads: http://mail.openjdk.java.net/pipermail/build-dev/2015-January/014064.html http://mail.openjdk.java.net/pipermail/build-dev/2015-February/014267.html The jdk9 changesets are here: http://hg.openjdk.java.net/jdk9/dev/rev/da879ffb91cc http://hg.openjdk.java.net/jdk9/dev/rev/507654772146 Unfortunately, the patches do not import cleanly. Here are the modified patches: diff -r d9ba714b6e7f -r 43ed2e8bd6ea common/autoconf/configure --- a/common/autoconf/configure Mon Mar 09 14:20:35 2015 -0400 +++ b/common/autoconf/configure Mon Mar 09 20:29:10 2015 -0400 @@ -28,6 +28,9 @@ exit 1 fi +# Force autoconf to use bash. +export CONFIG_SHELL=$BASH + CONFIGURE_COMMAND_LINE="$@" conf_script_dir=`dirname $0` diff -r 43ed2e8bd6ea -r b5121ccd1458 common/autoconf/configure --- a/common/autoconf/configure Mon Mar 09 20:29:10 2015 -0400 +++ b/common/autoconf/configure Mon Mar 09 20:30:23 2015 -0400 @@ -28,8 +28,9 @@ exit 1 fi -# Force autoconf to use bash. +# Force autoconf to use bash. This also means we must disable autoconf re-exec. export CONFIG_SHELL=$BASH +export _as_can_reexec=no CONFIGURE_COMMAND_LINE="$@" conf_script_dir=`dirname $0` Let me know if there are any concerns about these backports. thanks, dl From rob.mckenna at oracle.com Tue Mar 10 20:41:10 2015 From: rob.mckenna at oracle.com (Rob McKenna) Date: Tue, 10 Mar 2015 20:41:10 +0000 Subject: [8u60] Request for backport approval: 8069057 and 8072732 In-Reply-To: <54FF540B.8010601@oracle.com> References: <54FF540B.8010601@oracle.com> Message-ID: <54FF56E6.3090800@oracle.com> Hi Dean, if possible could you grab someone from build-dev to approve the (modest, I know) changes for 8? These are approved pending review, though please add the appropriate noreg keyword to the bugs. -Rob On 10/03/15 20:28, Dean Long wrote: > I would like to backport "8069057: Make sure configure is run by bash" > and the followup fix "8072732: Regression in configure due to > JDK-8069057" to 8u60. Both are needed by the previously approved > backport of "8064357: AARCH64: Top-level JDK changes". Without it, > JPRT fails when trying to use /bin/sh instead of /bin/bash. > > Bugs: > > https://bugs.openjdk.java.net/browse/JDK-8069057 > https://bugs.openjdk.java.net/browse/JDK-8072732 > > Original review threads: > > http://mail.openjdk.java.net/pipermail/build-dev/2015-January/014064.html > http://mail.openjdk.java.net/pipermail/build-dev/2015-February/014267.html > > > The jdk9 changesets are here: > > http://hg.openjdk.java.net/jdk9/dev/rev/da879ffb91cc > http://hg.openjdk.java.net/jdk9/dev/rev/507654772146 > > Unfortunately, the patches do not import cleanly. Here are the > modified patches: > > diff -r d9ba714b6e7f -r 43ed2e8bd6ea common/autoconf/configure > --- a/common/autoconf/configure Mon Mar 09 14:20:35 2015 -0400 > +++ b/common/autoconf/configure Mon Mar 09 20:29:10 2015 -0400 > @@ -28,6 +28,9 @@ > exit 1 > fi > > +# Force autoconf to use bash. > +export CONFIG_SHELL=$BASH > + > CONFIGURE_COMMAND_LINE="$@" > conf_script_dir=`dirname $0` > > diff -r 43ed2e8bd6ea -r b5121ccd1458 common/autoconf/configure > --- a/common/autoconf/configure Mon Mar 09 20:29:10 2015 -0400 > +++ b/common/autoconf/configure Mon Mar 09 20:30:23 2015 -0400 > @@ -28,8 +28,9 @@ > exit 1 > fi > > -# Force autoconf to use bash. > +# Force autoconf to use bash. This also means we must disable > autoconf re-exec. > export CONFIG_SHELL=$BASH > +export _as_can_reexec=no > > CONFIGURE_COMMAND_LINE="$@" > conf_script_dir=`dirname $0` > > Let me know if there are any concerns about > these backports. > > thanks, > > dl From rob.mckenna at oracle.com Tue Mar 10 20:41:34 2015 From: rob.mckenna at oracle.com (Rob McKenna) Date: Tue, 10 Mar 2015 20:41:34 +0000 Subject: [8u-dev] Request for Approval to backport JDK-8066436: Minimize can cause window to disappear on osx In-Reply-To: <54FF4058.3030200@oracle.com> References: <54FF4058.3030200@oracle.com> Message-ID: <54FF56FE.9020601@oracle.com> Approved. -Rob On 10/03/15 19:04, Nakul Natu wrote: > Hello, > > Please approve backporting this to 8u-dev. Patch applies cleanly after > paths are corrected for jdk8 > > Bug: https://bugs.openjdk.java.net/browse/JDK-8066436 > Webrev: /http://cr.openjdk.java.net/~van/8066436/webrev.01/ > / > Review thread: > http://mail.openjdk.java.net/pipermail/awt-dev/2015-March/009018.html > > Regards, > Nakul From alejandro.murillo at oracle.com Tue Mar 10 21:12:40 2015 From: alejandro.murillo at oracle.com (Alejandro E Murillo) Date: Tue, 10 Mar 2015 15:12:40 -0600 Subject: jdk8u60-b06: HotSpot Message-ID: <54FF5E48.3040206@oracle.com> hs25.60-b06 has been integrated into jdk8u60-b06. http://hg.openjdk.java.net/jdk8u/jdk8u/rev/74dbdbcb6aac http://hg.openjdk.java.net/jdk8u/jdk8u/corba/rev/587b01196646 http://hg.openjdk.java.net/jdk8u/jdk8u/hotspot/rev/7b70923c8e04 http://hg.openjdk.java.net/jdk8u/jdk8u/jaxp/rev/b265b3944437 http://hg.openjdk.java.net/jdk8u/jdk8u/jaxws/rev/a22a9460d53f http://hg.openjdk.java.net/jdk8u/jdk8u/jdk/rev/3a8ecea921f6 http://hg.openjdk.java.net/jdk8u/jdk8u/langtools/rev/44d168f9ad16 http://hg.openjdk.java.net/jdk8u/jdk8u/nashorn/rev/4b7613f08fd3 Component : VM Status : 0 major failures, 0 minor failure Date : 03/10/2015 at 12:00 MSK Tested By : STT_VM Cost(total man-days): 1 Workspace : 2015-03-06-144629.amurillo.hs25-60-b06-snapshot Bundles : 2015-03-06-144629.amurillo.hs25-60-b06-snapshot Platforms : Others Tests : Log : link Browsers : NA Patches : NA Number of Tests Executed : 390528 passed tests, 3804 failed tests (no new failures) Bug verification status: ====================================== Tested, Pass: Tested, Pass (partial fixes): Tested, Fail: Untested bug fixes: 7176220: 'Full GC' events miss date stamp information occasionally 8025667: Warning from b62 for hotspot.agent.src.os.solaris.proc: use after free 8049049: Unportable format string argument mismatch in hotspot/agent/src/os/solaris/proc/saproc.cpp 8073944: Simplify ArgumentsExt and remove unneeded functionallity 8074010: followup to 8072383 8074038: new hotspot build - hs25.60-b06 New bugs filed: Bugs in PIT build: Bugs in earlier promoted build: Number of PIT requested: 1 Integration target J2SE build number: jdk8u60-b06 Issues and Notes: This is PIT for HS25.60-b06 for jdk8u60-b06. Go for integration. -- Alejandro From Sergey.Bylokhov at oracle.com Tue Mar 10 21:40:18 2015 From: Sergey.Bylokhov at oracle.com (Sergey Bylokhov) Date: Tue, 10 Mar 2015 14:40:18 -0700 Subject: Result: New JDK 8u Committer: Alexander Zvegintsev Message-ID: <54FF64C2.1010406@oracle.com> Voting for Alexander Zvegintsev [1] is now closed. Yes: 8 Veto: 0 Abstain: 0 According to the Bylaws definition of Lazy Consensus, this is sufficient to approve the nomination. [1] http://mail.openjdk.java.net/pipermail/jdk8u-dev/2015-February/003096.html -- Best regards, Sergey. From dean.long at oracle.com Tue Mar 10 22:11:24 2015 From: dean.long at oracle.com (Dean Long) Date: Tue, 10 Mar 2015 15:11:24 -0700 Subject: [8u60] Request for backport approval: 8069057 and 8072732 In-Reply-To: <54FF56E6.3090800@oracle.com> References: <54FF540B.8010601@oracle.com> <54FF56E6.3090800@oracle.com> Message-ID: <54FF6C0C.80008@oracle.com> Magnus, could you double-check these changes? thanks, dl On 3/10/2015 1:41 PM, Rob McKenna wrote: > Hi Dean, if possible could you grab someone from build-dev to approve > the (modest, I know) changes for 8? > > These are approved pending review, though please add the appropriate > noreg keyword to the bugs. > > -Rob > > On 10/03/15 20:28, Dean Long wrote: >> I would like to backport "8069057: Make sure configure is run by >> bash" and the followup fix "8072732: Regression in configure due to >> JDK-8069057" to 8u60. Both are needed by the previously approved >> backport of "8064357: AARCH64: Top-level JDK changes". Without it, >> JPRT fails when trying to use /bin/sh instead of /bin/bash. >> >> Bugs: >> >> https://bugs.openjdk.java.net/browse/JDK-8069057 >> https://bugs.openjdk.java.net/browse/JDK-8072732 >> >> Original review threads: >> >> http://mail.openjdk.java.net/pipermail/build-dev/2015-January/014064.html >> >> http://mail.openjdk.java.net/pipermail/build-dev/2015-February/014267.html >> >> >> The jdk9 changesets are here: >> >> http://hg.openjdk.java.net/jdk9/dev/rev/da879ffb91cc >> http://hg.openjdk.java.net/jdk9/dev/rev/507654772146 >> >> Unfortunately, the patches do not import cleanly. Here are the >> modified patches: >> >> diff -r d9ba714b6e7f -r 43ed2e8bd6ea common/autoconf/configure >> --- a/common/autoconf/configure Mon Mar 09 14:20:35 2015 -0400 >> +++ b/common/autoconf/configure Mon Mar 09 20:29:10 2015 -0400 >> @@ -28,6 +28,9 @@ >> exit 1 >> fi >> >> +# Force autoconf to use bash. >> +export CONFIG_SHELL=$BASH >> + >> CONFIGURE_COMMAND_LINE="$@" >> conf_script_dir=`dirname $0` >> >> diff -r 43ed2e8bd6ea -r b5121ccd1458 common/autoconf/configure >> --- a/common/autoconf/configure Mon Mar 09 20:29:10 2015 -0400 >> +++ b/common/autoconf/configure Mon Mar 09 20:30:23 2015 -0400 >> @@ -28,8 +28,9 @@ >> exit 1 >> fi >> >> -# Force autoconf to use bash. >> +# Force autoconf to use bash. This also means we must disable >> autoconf re-exec. >> export CONFIG_SHELL=$BASH >> +export _as_can_reexec=no >> >> CONFIGURE_COMMAND_LINE="$@" >> conf_script_dir=`dirname $0` >> >> Let me know if there are any concerns about >> these backports. >> >> thanks, >> >> dl > From masayoshi.okutsu at oracle.com Wed Mar 11 08:46:31 2015 From: masayoshi.okutsu at oracle.com (Masayoshi Okutsu) Date: Wed, 11 Mar 2015 17:46:31 +0900 Subject: [8u60] Request for approval: 8074791: Long-form date format incorrect month string for Finnish locale Message-ID: <550000E7.5090906@oracle.com> Hello, Could you please approve backport of the fix for 8074791 to jdk8u-dev? The fix is identical to the JDK 9 one. Bug: https://bugs.openjdk.java.net/browse/JDK-8074791 Webrev: http://cr.openjdk.java.net/~okutsu/8u/8074791/webrev.00/ Review thread: http://mail.openjdk.java.net/pipermail/i18n-dev/2015-March/001637.html JDK9 changeset: http://hg.openjdk.java.net/jdk9/dev/jdk/rev/bacdcdbf8c02 Thanks, Masayoshi From erik.joelsson at oracle.com Wed Mar 11 08:48:09 2015 From: erik.joelsson at oracle.com (Erik Joelsson) Date: Wed, 11 Mar 2015 09:48:09 +0100 Subject: [8u60] Request for backport approval: 8069057 and 8072732 In-Reply-To: <54FF540B.8010601@oracle.com> References: <54FF540B.8010601@oracle.com> Message-ID: <55000149.1050709@oracle.com> The new patches look fine to me. /Erik On 2015-03-10 21:28, Dean Long wrote: > I would like to backport "8069057: Make sure configure is run by bash" > and the followup fix "8072732: Regression in configure due to > JDK-8069057" to 8u60. Both are needed by the previously approved > backport of "8064357: AARCH64: Top-level JDK changes". Without it, > JPRT fails when trying to use /bin/sh instead of /bin/bash. > > Bugs: > > https://bugs.openjdk.java.net/browse/JDK-8069057 > https://bugs.openjdk.java.net/browse/JDK-8072732 > > Original review threads: > > http://mail.openjdk.java.net/pipermail/build-dev/2015-January/014064.html > http://mail.openjdk.java.net/pipermail/build-dev/2015-February/014267.html > > > The jdk9 changesets are here: > > http://hg.openjdk.java.net/jdk9/dev/rev/da879ffb91cc > http://hg.openjdk.java.net/jdk9/dev/rev/507654772146 > > Unfortunately, the patches do not import cleanly. Here are the > modified patches: > > diff -r d9ba714b6e7f -r 43ed2e8bd6ea common/autoconf/configure > --- a/common/autoconf/configure Mon Mar 09 14:20:35 2015 -0400 > +++ b/common/autoconf/configure Mon Mar 09 20:29:10 2015 -0400 > @@ -28,6 +28,9 @@ > exit 1 > fi > > +# Force autoconf to use bash. > +export CONFIG_SHELL=$BASH > + > CONFIGURE_COMMAND_LINE="$@" > conf_script_dir=`dirname $0` > > diff -r 43ed2e8bd6ea -r b5121ccd1458 common/autoconf/configure > --- a/common/autoconf/configure Mon Mar 09 20:29:10 2015 -0400 > +++ b/common/autoconf/configure Mon Mar 09 20:30:23 2015 -0400 > @@ -28,8 +28,9 @@ > exit 1 > fi > > -# Force autoconf to use bash. > +# Force autoconf to use bash. This also means we must disable > autoconf re-exec. > export CONFIG_SHELL=$BASH > +export _as_can_reexec=no > > CONFIGURE_COMMAND_LINE="$@" > conf_script_dir=`dirname $0` > > Let me know if there are any concerns about > these backports. > > thanks, > > dl From sean.coffey at oracle.com Wed Mar 11 09:04:49 2015 From: sean.coffey at oracle.com (=?UTF-8?B?U2XDoW4gQ29mZmV5?=) Date: Wed, 11 Mar 2015 09:04:49 +0000 Subject: [8u60] Request for approval: 8074791: Long-form date format incorrect month string for Finnish locale In-Reply-To: <550000E7.5090906@oracle.com> References: <550000E7.5090906@oracle.com> Message-ID: <55000531.9080704@oracle.com> Approved. regards, Sean. On 11/03/2015 08:46, Masayoshi Okutsu wrote: > Hello, > > Could you please approve backport of the fix for 8074791 to jdk8u-dev? > The fix is identical to the JDK 9 one. > > Bug: > https://bugs.openjdk.java.net/browse/JDK-8074791 > Webrev: > http://cr.openjdk.java.net/~okutsu/8u/8074791/webrev.00/ > Review thread: > http://mail.openjdk.java.net/pipermail/i18n-dev/2015-March/001637.html > JDK9 changeset: > http://hg.openjdk.java.net/jdk9/dev/jdk/rev/bacdcdbf8c02 > > Thanks, > Masayoshi > From alexander.zvegintsev at oracle.com Wed Mar 11 12:28:31 2015 From: alexander.zvegintsev at oracle.com (Alexander Zvegintsev) Date: Wed, 11 Mar 2015 15:28:31 +0300 Subject: [8u60] Request for approval 8048289 Gtk: call to UIManager.getSystemLookAndFeelClassName() leads to crash Message-ID: <550034EF.6080009@oracle.com> Hello, Could you please approve a direct backport Bug: https://bugs.openjdk.java.net/browse/JDK-8048289 Fix: http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/dfcf4b835abd Review thread: http://mail.openjdk.java.net/pipermail/awt-dev/2014-July/008230.html -- Thanks, Alexander. From sean.coffey at oracle.com Wed Mar 11 12:33:16 2015 From: sean.coffey at oracle.com (=?UTF-8?B?U2XDoW4gQ29mZmV5?=) Date: Wed, 11 Mar 2015 12:33:16 +0000 Subject: [8u60] Request for approval 8048289 Gtk: call to UIManager.getSystemLookAndFeelClassName() leads to crash In-Reply-To: <550034EF.6080009@oracle.com> References: <550034EF.6080009@oracle.com> Message-ID: <5500360C.6040506@oracle.com> Please add a suitable noreg- label. Approved. regards, Sean. On 11/03/15 12:28, Alexander Zvegintsev wrote: > Hello, > > Could you please approve a direct backport > > Bug: > https://bugs.openjdk.java.net/browse/JDK-8048289 > Fix: > http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/dfcf4b835abd > Review thread: > http://mail.openjdk.java.net/pipermail/awt-dev/2014-July/008230.html > From attila.szegedi at oracle.com Wed Mar 11 12:42:29 2015 From: attila.szegedi at oracle.com (Attila Szegedi) Date: Wed, 11 Mar 2015 13:42:29 +0100 Subject: [8u60] Request for Approval: 8072426: Can't compare Java objects to strings or numbers Message-ID: Please approve. Bug: https://bugs.openjdk.java.net/browse/JDK-8072426 jdk9 webrev: http://cr.openjdk.java.net/~attila/8072426/webrev.00 jdk9 review thread: http://mail.openjdk.java.net/pipermail/nashorn-dev/2015-February/004191.html In order to backport this to 8u-dev, I had to make some changes. Namely, since we introduced a default method into the JSObject interface, we could not compile this under jdk8u-dev, as for JDK 8 Nashorn is compiled with "-source 7" as the part of the build process is running Nasgen, which is done with the boot JDK, which is a JDK 7 for 8u? For this reason, the 8u backport doesn't add a new getDefaultValue() method to the JSObject interface, as that ? without the default methods' availability with "-source 7" ? wouldn't be a binary compatible class change. Instead, AbstractJSObject gains a getDefaultValue(hint) method with the same behavior as well as a static helper getDefaultValue(jsobj, hint) that delegates to AbstractJSObject.getDefaultValue() for AbstractJSObject, and runs the generic algorithm for any other JSObject. (As a separate task, I will soon be forward-porting this static helper into JDK 9 codebase so it stays compatible with 8; JDK-8074661 is filed for that.) jdk8 webrev: http://cr.openjdk.java.net/~attila/8072426/webrev.8u jdk8 review thread: http://mail.openjdk.java.net/pipermail/nashorn-dev/2015-March/004273.html Thanks, Attila. From sean.coffey at oracle.com Wed Mar 11 13:12:19 2015 From: sean.coffey at oracle.com (=?windows-1252?Q?Se=E1n_Coffey?=) Date: Wed, 11 Mar 2015 13:12:19 +0000 Subject: [8u60] Request for Approval: 8072426: Can't compare Java objects to strings or numbers In-Reply-To: References: Message-ID: <55003F33.8000405@oracle.com> Approved. regards, Sean. On 11/03/15 12:42, Attila Szegedi wrote: > Please approve. > > Bug: https://bugs.openjdk.java.net/browse/JDK-8072426 > jdk9 webrev: http://cr.openjdk.java.net/~attila/8072426/webrev.00 > jdk9 review thread: http://mail.openjdk.java.net/pipermail/nashorn-dev/2015-February/004191.html > > In order to backport this to 8u-dev, I had to make some changes. Namely, since we introduced a default method into the JSObject interface, we could not compile this under jdk8u-dev, as for JDK 8 Nashorn is compiled with "-source 7" as the part of the build process is running Nasgen, which is done with the boot JDK, which is a JDK 7 for 8u? > > For this reason, the 8u backport doesn't add a new getDefaultValue() method to the JSObject interface, as that ? without the default methods' availability with "-source 7" ? wouldn't be a binary compatible class change. Instead, AbstractJSObject gains a getDefaultValue(hint) method with the same behavior as well as a static helper getDefaultValue(jsobj, hint) that delegates to AbstractJSObject.getDefaultValue() for AbstractJSObject, and runs the generic algorithm for any other JSObject. (As a separate task, I will soon be forward-porting this static helper into JDK 9 codebase so it stays compatible with 8; JDK-8074661 is filed for that.) > > jdk8 webrev: http://cr.openjdk.java.net/~attila/8072426/webrev.8u > jdk8 review thread: http://mail.openjdk.java.net/pipermail/nashorn-dev/2015-March/004273.html > > Thanks, > Attila. From attila.szegedi at oracle.com Wed Mar 11 16:54:07 2015 From: attila.szegedi at oracle.com (Attila Szegedi) Date: Wed, 11 Mar 2015 17:54:07 +0100 Subject: [8u60] Request for Approval: 8074031: Canonicalize is-a-JS-string tests Message-ID: <81B36855-48EF-44FA-BC52-D0181A5B197A@oracle.com> Please approve. Bug: https://bugs.openjdk.java.net/browse/JDK-8074031 jdk9 webrev: http://cr.openjdk.java.net/~attila/8074031/webrev.00 jdk9 review thread: http://mail.openjdk.java.net/pipermail/nashorn-dev/2015-February/004230.html Changes apply cleanly to jdk8u-dev after path reshuffling from modular source code layout. Thanks, Attila. From attila.szegedi at oracle.com Wed Mar 11 16:54:57 2015 From: attila.szegedi at oracle.com (Attila Szegedi) Date: Wed, 11 Mar 2015 17:54:57 +0100 Subject: [8u60] Request for Approval: 8035712: Restore some of the RuntimeCallSite specializations Message-ID: <75B6BBE9-CF79-4799-B48A-B6A0EEC59152@oracle.com> Please approve. Bug: https://bugs.openjdk.java.net/browse/JDK-8035712 jdk9 webrev: http://cr.openjdk.java.net/~attila/8035712/webrev.00 jdk9 review thread: http://mail.openjdk.java.net/pipermail/nashorn-dev/2015-March/004246.html Changes apply cleanly to jdk8u-dev after path reshuffling from modular source code layout. Thanks, Attila. From attila.szegedi at oracle.com Wed Mar 11 16:55:40 2015 From: attila.szegedi at oracle.com (Attila Szegedi) Date: Wed, 11 Mar 2015 17:55:40 +0100 Subject: [8u60] Request for Approval: 8074487: Static analysis of IfNode should consider terminating branches Message-ID: <8C010BEB-ECCC-4E0F-A544-CCC23BDC789E@oracle.com> Please approve. Bug: https://bugs.openjdk.java.net/browse/JDK-8074487 jdk9 webrev: http://cr.openjdk.java.net/~attila/8074487/webrev.00 jdk9 review thread: http://mail.openjdk.java.net/pipermail/nashorn-dev/2015-March/004260.html Changes apply cleanly to jdk8u-dev after path reshuffling from modular source code layout. Thanks, Attila. From attila.szegedi at oracle.com Wed Mar 11 16:55:54 2015 From: attila.szegedi at oracle.com (Attila Szegedi) Date: Wed, 11 Mar 2015 17:55:54 +0100 Subject: [8u60] Request for Approval: 8074484: More agressive value discarding Message-ID: <72BF070C-E49B-48CD-8C2C-09C25E92369B@oracle.com> Please approve. Bug: https://bugs.openjdk.java.net/browse/JDK-8074484 jdk9 webrev: http://cr.openjdk.java.net/~attila/8074484/webrev.00 jdk9 review thread: http://mail.openjdk.java.net/pipermail/nashorn-dev/2015-March/004264.html Changes apply cleanly to jdk8u-dev after path reshuffling from modular source code layout. Thanks, Attila. From dean.long at oracle.com Wed Mar 11 17:44:05 2015 From: dean.long at oracle.com (Dean Long) Date: Wed, 11 Mar 2015 10:44:05 -0700 Subject: [8u60] Request for backport approval: 8069057 and 8072732 In-Reply-To: <55000149.1050709@oracle.com> References: <54FF540B.8010601@oracle.com> <55000149.1050709@oracle.com> Message-ID: <55007EE5.6010606@oracle.com> Thanks Erik. dl On 3/11/2015 1:48 AM, Erik Joelsson wrote: > The new patches look fine to me. > > /Erik > > On 2015-03-10 21:28, Dean Long wrote: >> I would like to backport "8069057: Make sure configure is run by >> bash" and the followup fix "8072732: Regression in configure due to >> JDK-8069057" to 8u60. Both are needed by the previously approved >> backport of "8064357: AARCH64: Top-level JDK changes". Without it, >> JPRT fails when trying to use /bin/sh instead of /bin/bash. >> >> Bugs: >> >> https://bugs.openjdk.java.net/browse/JDK-8069057 >> https://bugs.openjdk.java.net/browse/JDK-8072732 >> >> Original review threads: >> >> http://mail.openjdk.java.net/pipermail/build-dev/2015-January/014064.html >> >> http://mail.openjdk.java.net/pipermail/build-dev/2015-February/014267.html >> >> >> The jdk9 changesets are here: >> >> http://hg.openjdk.java.net/jdk9/dev/rev/da879ffb91cc >> http://hg.openjdk.java.net/jdk9/dev/rev/507654772146 >> >> Unfortunately, the patches do not import cleanly. Here are the >> modified patches: >> >> diff -r d9ba714b6e7f -r 43ed2e8bd6ea common/autoconf/configure >> --- a/common/autoconf/configure Mon Mar 09 14:20:35 2015 -0400 >> +++ b/common/autoconf/configure Mon Mar 09 20:29:10 2015 -0400 >> @@ -28,6 +28,9 @@ >> exit 1 >> fi >> >> +# Force autoconf to use bash. >> +export CONFIG_SHELL=$BASH >> + >> CONFIGURE_COMMAND_LINE="$@" >> conf_script_dir=`dirname $0` >> >> diff -r 43ed2e8bd6ea -r b5121ccd1458 common/autoconf/configure >> --- a/common/autoconf/configure Mon Mar 09 20:29:10 2015 -0400 >> +++ b/common/autoconf/configure Mon Mar 09 20:30:23 2015 -0400 >> @@ -28,8 +28,9 @@ >> exit 1 >> fi >> >> -# Force autoconf to use bash. >> +# Force autoconf to use bash. This also means we must disable >> autoconf re-exec. >> export CONFIG_SHELL=$BASH >> +export _as_can_reexec=no >> >> CONFIGURE_COMMAND_LINE="$@" >> conf_script_dir=`dirname $0` >> >> Let me know if there are any concerns about >> these backports. >> >> thanks, >> >> dl > From rob.mckenna at oracle.com Wed Mar 11 18:03:50 2015 From: rob.mckenna at oracle.com (Rob McKenna) Date: Wed, 11 Mar 2015 18:03:50 +0000 Subject: [8u60] Request for Approval: 8074031: Canonicalize is-a-JS-string tests In-Reply-To: <81B36855-48EF-44FA-BC52-D0181A5B197A@oracle.com> References: <81B36855-48EF-44FA-BC52-D0181A5B197A@oracle.com> Message-ID: <55008386.4040508@oracle.com> Approved. -Rob On 11/03/15 16:54, Attila Szegedi wrote: > Please approve. > > Bug: https://bugs.openjdk.java.net/browse/JDK-8074031 > jdk9 webrev: http://cr.openjdk.java.net/~attila/8074031/webrev.00 > jdk9 review thread: http://mail.openjdk.java.net/pipermail/nashorn-dev/2015-February/004230.html > > Changes apply cleanly to jdk8u-dev after path reshuffling from modular source code layout. > > Thanks, > Attila. From rob.mckenna at oracle.com Wed Mar 11 18:04:17 2015 From: rob.mckenna at oracle.com (Rob McKenna) Date: Wed, 11 Mar 2015 18:04:17 +0000 Subject: [8u60] Request for Approval: 8035712: Restore some of the RuntimeCallSite specializations In-Reply-To: <75B6BBE9-CF79-4799-B48A-B6A0EEC59152@oracle.com> References: <75B6BBE9-CF79-4799-B48A-B6A0EEC59152@oracle.com> Message-ID: <550083A1.8060407@oracle.com> Approved. -Rob On 11/03/15 16:54, Attila Szegedi wrote: > Please approve. > > Bug: https://bugs.openjdk.java.net/browse/JDK-8035712 > jdk9 webrev: http://cr.openjdk.java.net/~attila/8035712/webrev.00 > jdk9 review thread: http://mail.openjdk.java.net/pipermail/nashorn-dev/2015-March/004246.html > > Changes apply cleanly to jdk8u-dev after path reshuffling from modular source code layout. > > Thanks, > Attila. From rob.mckenna at oracle.com Wed Mar 11 18:04:38 2015 From: rob.mckenna at oracle.com (Rob McKenna) Date: Wed, 11 Mar 2015 18:04:38 +0000 Subject: [8u60] Request for Approval: 8074487: Static analysis of IfNode should consider terminating branches In-Reply-To: <8C010BEB-ECCC-4E0F-A544-CCC23BDC789E@oracle.com> References: <8C010BEB-ECCC-4E0F-A544-CCC23BDC789E@oracle.com> Message-ID: <550083B6.5030304@oracle.com> Approved. -Rob On 11/03/15 16:55, Attila Szegedi wrote: > Please approve. > > Bug: https://bugs.openjdk.java.net/browse/JDK-8074487 > jdk9 webrev: http://cr.openjdk.java.net/~attila/8074487/webrev.00 > jdk9 review thread: http://mail.openjdk.java.net/pipermail/nashorn-dev/2015-March/004260.html > > Changes apply cleanly to jdk8u-dev after path reshuffling from modular source code layout. > > Thanks, > Attila. From rob.mckenna at oracle.com Wed Mar 11 18:05:01 2015 From: rob.mckenna at oracle.com (Rob McKenna) Date: Wed, 11 Mar 2015 18:05:01 +0000 Subject: [8u60] Request for Approval: 8074484: More agressive value discarding In-Reply-To: <72BF070C-E49B-48CD-8C2C-09C25E92369B@oracle.com> References: <72BF070C-E49B-48CD-8C2C-09C25E92369B@oracle.com> Message-ID: <550083CD.4010900@oracle.com> Approved. -Rob On 11/03/15 16:55, Attila Szegedi wrote: > Please approve. > > Bug: https://bugs.openjdk.java.net/browse/JDK-8074484 > jdk9 webrev: http://cr.openjdk.java.net/~attila/8074484/webrev.00 > jdk9 review thread: http://mail.openjdk.java.net/pipermail/nashorn-dev/2015-March/004264.html > > Changes apply cleanly to jdk8u-dev after path reshuffling from modular source code layout. > > Thanks, > Attila. From dean.long at oracle.com Wed Mar 11 21:14:50 2015 From: dean.long at oracle.com (Dean Long) Date: Wed, 11 Mar 2015 14:14:50 -0700 Subject: [8u60] Request for backport approval: 8068927: AARCH64: better handling of aarch64- triples Message-ID: <5500B04A.5080600@oracle.com> I would like to backport "8068927: AARCH64: better handling of aarch64- triples" to 8u60. This is a followup to "8064357: AARCH64: Top-level JDK changes". Bug: https://bugs.openjdk.java.net/browse/JDK-8068927 Original review thread: http://mail.openjdk.java.net/pipermail/build-dev/2015-January/014052.html The jdk9 changeset is here: http://hg.openjdk.java.net/jdk9/dev/rev/c360aeb77273 The patch imported cleanly. Let me know if there are any concerns about this backport. thanks, dl From jaroslav.bachorik at oracle.com Thu Mar 12 08:41:08 2015 From: jaroslav.bachorik at oracle.com (Jaroslav Bachorik) Date: Thu, 12 Mar 2015 09:41:08 +0100 Subject: [8u-dev] Request for approval: 8071657 - JDI ObjectReferenceImpl.invokeMethod() validation fails for virtual invocations of method with declaring type being an interface Message-ID: <55015124.6000809@oracle.com> Please, approve this backport of a JDK 9 fix. Issue : https://bugs.openjdk.java.net/browse/JDK-8071657 Changeset : http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/9c769c3a2c60 Review : http://mail.openjdk.java.net/pipermail/serviceability-dev/2015-February/016626.html The patch applies cleanly after reshuffling from the modular layout. Thanks, -JB- From ivan.gerasimov at oracle.com Thu Mar 12 08:59:41 2015 From: ivan.gerasimov at oracle.com (Ivan Gerasimov) Date: Thu, 12 Mar 2015 11:59:41 +0300 Subject: [8u-dev] Request for Review + Request for Approval to Backport: Table's field width in "Use" page generated by javadoc with '-s' is unbalanced Message-ID: <5501557D.1080107@oracle.com> Hello! Would you please approve the backport of this fix to 8u-dev? The source fix applies cleanly after unshufling. The test needed some minor adjustments, thus the Request for Review. Bug: https://bugs.openjdk.java.net/browse/JDK-8072461 Jdk9 changeset: http://hg.openjdk.java.net/jdk9/dev/langtools/rev/55ce0df1fb75 Jdk8u webrev: http://cr.openjdk.java.net/~igerasim/8072461/0/webrev/ Sincerely yours, Ivan From sean.coffey at oracle.com Thu Mar 12 09:20:06 2015 From: sean.coffey at oracle.com (=?windows-1252?Q?Se=E1n_Coffey?=) Date: Thu, 12 Mar 2015 09:20:06 +0000 Subject: [8u-dev] Request for approval: 8071657 - JDI ObjectReferenceImpl.invokeMethod() validation fails for virtual invocations of method with declaring type being an interface In-Reply-To: <55015124.6000809@oracle.com> References: <55015124.6000809@oracle.com> Message-ID: <55015A46.4060402@oracle.com> Approved. regards, Sean. On 12/03/2015 08:41, Jaroslav Bachorik wrote: > Please, approve this backport of a JDK 9 fix. > > Issue : https://bugs.openjdk.java.net/browse/JDK-8071657 > Changeset : http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/9c769c3a2c60 > Review : > http://mail.openjdk.java.net/pipermail/serviceability-dev/2015-February/016626.html > > The patch applies cleanly after reshuffling from the modular layout. > > Thanks, > > -JB- From sean.coffey at oracle.com Thu Mar 12 09:25:38 2015 From: sean.coffey at oracle.com (=?UTF-8?B?U2XDoW4gQ29mZmV5?=) Date: Thu, 12 Mar 2015 09:25:38 +0000 Subject: [8u-dev] Request for Review + Request for Approval to Backport: Table's field width in "Use" page generated by javadoc with '-s' is unbalanced In-Reply-To: <5501557D.1080107@oracle.com> References: <5501557D.1080107@oracle.com> Message-ID: <55015B92.8050805@oracle.com> test changes look harmless to me. Approved. Can you open up the bug report to public also ? thanks, Sean. On 12/03/2015 08:59, Ivan Gerasimov wrote: > Hello! > > Would you please approve the backport of this fix to 8u-dev? > The source fix applies cleanly after unshufling. > The test needed some minor adjustments, thus the Request for Review. > > Bug: https://bugs.openjdk.java.net/browse/JDK-8072461 > Jdk9 changeset: > http://hg.openjdk.java.net/jdk9/dev/langtools/rev/55ce0df1fb75 > Jdk8u webrev: http://cr.openjdk.java.net/~igerasim/8072461/0/webrev/ > > Sincerely yours, > Ivan From sean.coffey at oracle.com Thu Mar 12 09:28:27 2015 From: sean.coffey at oracle.com (=?UTF-8?B?U2XDoW4gQ29mZmV5?=) Date: Thu, 12 Mar 2015 09:28:27 +0000 Subject: [8u60] Request for backport approval: 8068927: AARCH64: better handling of aarch64- triples In-Reply-To: <5500B04A.5080600@oracle.com> References: <5500B04A.5080600@oracle.com> Message-ID: <55015C3B.4080207@oracle.com> Approved. regards, Sean. On 11/03/2015 21:14, Dean Long wrote: > I would like to backport "8068927: AARCH64: better handling of > aarch64- triples" to 8u60. This is a followup to "8064357: AARCH64: > Top-level JDK changes". > > Bug: > > https://bugs.openjdk.java.net/browse/JDK-8068927 > > Original review thread: > > http://mail.openjdk.java.net/pipermail/build-dev/2015-January/014052.html > > The jdk9 changeset is here: > > http://hg.openjdk.java.net/jdk9/dev/rev/c360aeb77273 > > The patch imported cleanly. Let me know if there are any concerns about > this backport. > > thanks, > > dl From Sergey.Bylokhov at oracle.com Thu Mar 12 16:15:22 2015 From: Sergey.Bylokhov at oracle.com (Sergey Bylokhov) Date: Thu, 12 Mar 2015 09:15:22 -0700 Subject: [8u-dev] Request for approval: 8074668 [macosx] Mac 10.10: Application run with splash screen has focus issues Message-ID: <5501BB9A.1060109@oracle.com> Hello, This is a direct back port from jdk 9 to jdk 8u-dev. Bug: https://bugs.openjdk.java.net/browse/JDK-8074668 Review: http://mail.openjdk.java.net/pipermail/2d-dev/2015-March/005181.html jdk9 changeset: http://hg.openjdk.java.net/jdk9/client/jdk/rev/edb18ea158ff Reviewers: Alexander Scherbatiy, Phil Race, Anton Tarasov. -- Best regards, Sergey. From sean.coffey at oracle.com Thu Mar 12 16:25:20 2015 From: sean.coffey at oracle.com (=?UTF-8?B?U2XDoW4gQ29mZmV5?=) Date: Thu, 12 Mar 2015 16:25:20 +0000 Subject: [8u-dev] Request for approval: 8074668 [macosx] Mac 10.10: Application run with splash screen has focus issues In-Reply-To: <5501BB9A.1060109@oracle.com> References: <5501BB9A.1060109@oracle.com> Message-ID: <5501BDF0.8030003@oracle.com> Approved. regards, Sean. On 12/03/2015 16:15, Sergey Bylokhov wrote: > Hello, > This is a direct back port from jdk 9 to jdk 8u-dev. > > Bug: https://bugs.openjdk.java.net/browse/JDK-8074668 > Review: > http://mail.openjdk.java.net/pipermail/2d-dev/2015-March/005181.html > jdk9 changeset: > http://hg.openjdk.java.net/jdk9/client/jdk/rev/edb18ea158ff > > Reviewers: Alexander Scherbatiy, Phil Race, Anton Tarasov. > From hannes.wallnoefer at oracle.com Thu Mar 12 16:37:26 2015 From: hannes.wallnoefer at oracle.com (Hannes Wallnoefer) Date: Thu, 12 Mar 2015 17:37:26 +0100 Subject: [8u60] Request for approval for JDK-8074693 and JDK-8074687 Message-ID: <5501C0C6.4000708@oracle.com> Please approve backport of two Nashorn bugs to 8u60: Bug: https://bugs.openjdk.java.net/browse/JDK-8074693 Webrev: http://cr.openjdk.java.net/~hannesw/8074693/ Review thread: http://mail.openjdk.java.net/pipermail/nashorn-dev/2015-March/004298.html Bug: https://bugs.openjdk.java.net/browse/JDK-8074687 Webrev: http://cr.openjdk.java.net/~hannesw/8074687/ Review thread: http://mail.openjdk.java.net/pipermail/nashorn-dev/2015-March/004299.html The changes apply cleanly to 8u-dev after reshuffling path names. Thanks, Hannes From rob.mckenna at oracle.com Thu Mar 12 16:54:42 2015 From: rob.mckenna at oracle.com (Rob McKenna) Date: Thu, 12 Mar 2015 16:54:42 +0000 Subject: [8u60] Request for approval for JDK-8074693 and JDK-8074687 In-Reply-To: <5501C0C6.4000708@oracle.com> References: <5501C0C6.4000708@oracle.com> Message-ID: <5501C4D2.2050404@oracle.com> Approved. -Rob On 12/03/15 16:37, Hannes Wallnoefer wrote: > Please approve backport of two Nashorn bugs to 8u60: > > Bug: https://bugs.openjdk.java.net/browse/JDK-8074693 > Webrev: http://cr.openjdk.java.net/~hannesw/8074693/ > Review thread: > http://mail.openjdk.java.net/pipermail/nashorn-dev/2015-March/004298.html > > Bug: https://bugs.openjdk.java.net/browse/JDK-8074687 > Webrev: http://cr.openjdk.java.net/~hannesw/8074687/ > Review thread: > http://mail.openjdk.java.net/pipermail/nashorn-dev/2015-March/004299.html > > The changes apply cleanly to 8u-dev after reshuffling path names. > > Thanks, > Hannes From alejandro.murillo at oracle.com Fri Mar 13 21:58:37 2015 From: alejandro.murillo at oracle.com (Alejandro E Murillo) Date: Fri, 13 Mar 2015 15:58:37 -0600 Subject: [8u60] Request for approval for bulk integration of hs25.60-b07 Message-ID: <55035D8D.7050100@oracle.com> Requesting approval to integrate hs25.60-b07 into jdk8u60-b07. A webrev is available at: http://cr.openjdk.java.net/~amurillo/8u60/hs25.60-b07-jdk8u60-b07.webrev/ Pre-integration testing is in progress; the integration will proceed only after SQE has analyzed the results and approved. The fixes in the proposed integration are below. All have undergone nightly testing and are already in a jdk9 repository. 8006960: hotspot, "impossible" assertion failure 8043224: -Xcheck:jni improvements to exception checking and excessive local refs 8071501: perfMemory_solaris.cpp failing to compile with "Error: dd_fd is not a member of DIR." 8073688: Infinite loop reading types during jmap attach. 8074550: new hotspot build - hs25.60-b07 8075158: Make jdk8u60 the default release on jdk8u repos -- Alejandro From sean.coffey at oracle.com Sat Mar 14 13:41:38 2015 From: sean.coffey at oracle.com (=?UTF-8?B?U2XDoW4gQ29mZmV5?=) Date: Sat, 14 Mar 2015 13:41:38 +0000 Subject: [8u60] Request for approval for bulk integration of hs25.60-b07 In-Reply-To: <55035D8D.7050100@oracle.com> References: <55035D8D.7050100@oracle.com> Message-ID: <55043A92.7080205@oracle.com> Approved pending positive PIT results. regards, Sean. On 13/03/2015 21:58, Alejandro E Murillo wrote: > Requesting approval to integrate hs25.60-b07 into jdk8u60-b07. > > A webrev is available at: > > http://cr.openjdk.java.net/~amurillo/8u60/hs25.60-b07-jdk8u60-b07.webrev/ > > Pre-integration testing is in progress; the integration will proceed > only after SQE has analyzed the results and approved. > > The fixes in the proposed integration are below. All have undergone > nightly testing and are already in a jdk9 repository. > > 8006960: hotspot, "impossible" assertion failure > 8043224: -Xcheck:jni improvements to exception checking and excessive > local refs > 8071501: perfMemory_solaris.cpp failing to compile with "Error: dd_fd > is not a member of DIR." > 8073688: Infinite loop reading types during jmap attach. > 8074550: new hotspot build - hs25.60-b07 > 8075158: Make jdk8u60 the default release on jdk8u repos > From attila.szegedi at oracle.com Mon Mar 16 10:04:30 2015 From: attila.szegedi at oracle.com (Attila Szegedi) Date: Mon, 16 Mar 2015 11:04:30 +0100 Subject: [8u60] Request for Approval: 8073706: Livelock in CompiledFunction.getValidOptimisticInvocation Message-ID: <113658C0-F70C-483A-BB57-118137D5DFA6@oracle.com> Please approve. Bug: https://bugs.openjdk.java.net/browse/JDK-8073706 jdk9 webrev: http://cr.openjdk.java.net/~attila/8073706/webrev.00 jdk9 review thread: http://mail.openjdk.java.net/pipermail/nashorn-dev/2015-March/004327.html Changes apply cleanly to jdk8u-dev after path reshuffling from modular source code layout. Thanks, Attila. From rob.mckenna at oracle.com Mon Mar 16 15:15:54 2015 From: rob.mckenna at oracle.com (Rob McKenna) Date: Mon, 16 Mar 2015 15:15:54 +0000 Subject: [8u60] Request for Approval: 8073706: Livelock in CompiledFunction.getValidOptimisticInvocation In-Reply-To: <113658C0-F70C-483A-BB57-118137D5DFA6@oracle.com> References: <113658C0-F70C-483A-BB57-118137D5DFA6@oracle.com> Message-ID: <5506F3AA.3090706@oracle.com> Approved. -Rob On 16/03/15 10:04, Attila Szegedi wrote: > Please approve. > > Bug: https://bugs.openjdk.java.net/browse/JDK-8073706 > jdk9 webrev: http://cr.openjdk.java.net/~attila/8073706/webrev.00 > jdk9 review thread: http://mail.openjdk.java.net/pipermail/nashorn-dev/2015-March/004327.html > > Changes apply cleanly to jdk8u-dev after path reshuffling from modular source code layout. > > Thanks, > Attila. From marcus.lagergren at oracle.com Mon Mar 16 15:22:48 2015 From: marcus.lagergren at oracle.com (Marcus Lagergren) Date: Mon, 16 Mar 2015 16:22:48 +0100 Subject: [8u60] Request for Approval: 8066217 : ArrayBuffer construction without length Message-ID: <01A539DA-0871-4769-BC58-C579C09728B9@oracle.com> Bug: https://bugs.openjdk.java.net/browse/JDK-8066217 jdk9 webrev: http://cr.openjdk.java.net/~lagergren/8066217/ jdk9 review thread: http://mail.openjdk.java.net/pipermail/nashorn-dev/2015-March/004347.html /M From rob.mckenna at oracle.com Mon Mar 16 15:24:59 2015 From: rob.mckenna at oracle.com (Rob McKenna) Date: Mon, 16 Mar 2015 15:24:59 +0000 Subject: [8u60] Request for Approval: 8066217 : ArrayBuffer construction without length In-Reply-To: <01A539DA-0871-4769-BC58-C579C09728B9@oracle.com> References: <01A539DA-0871-4769-BC58-C579C09728B9@oracle.com> Message-ID: <5506F5CB.1090207@oracle.com> Approved. -Rob On 16/03/15 15:22, Marcus Lagergren wrote: > Bug: https://bugs.openjdk.java.net/browse/JDK-8066217 > jdk9 webrev: http://cr.openjdk.java.net/~lagergren/8066217/ > jdk9 review thread: http://mail.openjdk.java.net/pipermail/nashorn-dev/2015-March/004347.html > > /M > From anton.nashatyrev at oracle.com Mon Mar 16 17:35:43 2015 From: anton.nashatyrev at oracle.com (Anton Nashatyrev) Date: Mon, 16 Mar 2015 20:35:43 +0300 Subject: [8u-dev] Backport request: 8073008: press-and-hold input method for accented characters works incorrectly on OS X Message-ID: <5507146F.6070306@oracle.com> Hello, please approve the fix backport from jdk9 to jdk8u-dev. The patch was applied strictly. Bug: https://bugs.openjdk.java.net/browse/JDK-8073008 Jdk9 Changeset: http://hg.openjdk.java.net/jdk9/client/jdk/rev/fc76d530de50 Jdk9 Review Thread: http://mail.openjdk.java.net/pipermail/awt-dev/2015-March/009037.html Thanks! Anton. From rob.mckenna at oracle.com Mon Mar 16 17:49:35 2015 From: rob.mckenna at oracle.com (Rob McKenna) Date: Mon, 16 Mar 2015 17:49:35 +0000 Subject: [8u-dev] Backport request: 8073008: press-and-hold input method for accented characters works incorrectly on OS X In-Reply-To: <5507146F.6070306@oracle.com> References: <5507146F.6070306@oracle.com> Message-ID: <550717AF.9040903@oracle.com> Approved. -Rob On 16/03/15 17:35, Anton Nashatyrev wrote: > Hello, > > please approve the fix backport from jdk9 to jdk8u-dev. The patch > was applied strictly. > > Bug: https://bugs.openjdk.java.net/browse/JDK-8073008 > Jdk9 Changeset: > http://hg.openjdk.java.net/jdk9/client/jdk/rev/fc76d530de50 > Jdk9 Review Thread: > http://mail.openjdk.java.net/pipermail/awt-dev/2015-March/009037.html > > Thanks! > Anton. From alejandro.murillo at oracle.com Tue Mar 17 18:38:09 2015 From: alejandro.murillo at oracle.com (Alejandro E Murillo) Date: Tue, 17 Mar 2015 11:38:09 -0700 (PDT) Subject: jdk8u60-b07: HotSpot Message-ID: <55087491.80006@oracle.com> hs25.60-b07 has been integrated into jdk8u60-b07. http://hg.openjdk.java.net/jdk8u/jdk8u/rev/ea714a39e902 http://hg.openjdk.java.net/jdk8u/jdk8u/corba/rev/058a6dd8d04c http://hg.openjdk.java.net/jdk8u/jdk8u/hotspot/rev/353e580ce687 http://hg.openjdk.java.net/jdk8u/jdk8u/jaxp/rev/51eab6899806 http://hg.openjdk.java.net/jdk8u/jdk8u/jaxws/rev/6f0885023e43 http://hg.openjdk.java.net/jdk8u/jdk8u/jdk/rev/e48ca20d8943 http://hg.openjdk.java.net/jdk8u/jdk8u/langtools/rev/39b47ffeb778 http://hg.openjdk.java.net/jdk8u/jdk8u/nashorn/rev/80966e5cc384 Component : VM Status : 0 major failures, 0 minor failure Date : 03/17/2015 at 18:00 MSK Tested By : STT_VM Cost(total man-days): 1 Workspace : 2015-03-13-194526.amurillo.hs25-60-b07-snapshot Bundles : 2015-03-13-194526.amurillo.hs25-60-b07-snapshot Platforms : Others Tests : Log : link Browsers : NA Patches : NA Number of Tests Executed : 421883 passed tests, 3794 failed tests (no new failures) Bug verification status: ====================================== Tested, Pass: Tested, Pass (partial fixes): Tested, Fail: Untested bug fixes: 8006960: hotspot, "impossible" assertion failure 8043224: -Xcheck:jni improvements to exception checking and excessive local refs 8071501: perfMemory_solaris.cpp failing to compile with "Error: dd_fd is not a member of DIR." 8073688: Infinite loop reading types during jmap attach. 8074550: new hotspot build - hs25.60-b07 8075158: Make jdk8u60 the default release on jdk8u repos New bugs filed: Bugs in PIT build: Bugs in earlier promoted build: Number of PIT requested: 1 Integration target J2SE build number: jdk8u60-b07 Issues and Notes: This is PIT for HS25.60-b07 for jdk8u60-b07. Go for integration. -- Alejandro From volker.simonis at gmail.com Wed Mar 18 10:53:08 2015 From: volker.simonis at gmail.com (Volker Simonis) Date: Wed, 18 Mar 2015 11:53:08 +0100 Subject: [8u60] request for approval: 8071687: AIX port of "8039173: Propagate errors from Diagnostic Commands as exceptions in the attach framework" Message-ID: Hi, could you please approve the backport of the following AIX-only fix to jd8u-dev: 8071687: AIX port of "8039173: Propagate errors from Diagnostic Commands as exceptions in the attach framework" Bug: https://bugs.openjdk.java.net/browse/JDK-8071687 Webrev (8u-dev): http://cr.openjdk.java.net/~simonis/webrevs/2015/8071687_8u/ Original Webrev (jdk9): http://cr.openjdk.java.net/~simonis/webrevs/2015/8071687/ Review:http://mail.openjdk.java.net/pipermail/serviceability-dev/2015-March/016933.html The webrev for 8u-dev just adapts the copyright headers and the source location (i.e. to before the "modular source change"). 8039173, which this change is a fix for on AIX already has been integrated into 8u a while ago. Thanks, Volker From sean.coffey at oracle.com Wed Mar 18 11:14:58 2015 From: sean.coffey at oracle.com (=?UTF-8?B?U2XDoW4gQ29mZmV5?=) Date: Wed, 18 Mar 2015 11:14:58 +0000 Subject: [8u60] request for approval: 8071687: AIX port of "8039173: Propagate errors from Diagnostic Commands as exceptions in the attach framework" In-Reply-To: References: Message-ID: <55095E32.40500@oracle.com> Approved. regards, Sean. On 18/03/15 10:53, Volker Simonis wrote: > Hi, > > could you please approve the backport of the following AIX-only fix to jd8u-dev: > > 8071687: AIX port of "8039173: Propagate errors from Diagnostic > Commands as exceptions in the attach framework" > > Bug: https://bugs.openjdk.java.net/browse/JDK-8071687 > Webrev (8u-dev): http://cr.openjdk.java.net/~simonis/webrevs/2015/8071687_8u/ > Original Webrev (jdk9): > http://cr.openjdk.java.net/~simonis/webrevs/2015/8071687/ > Review:http://mail.openjdk.java.net/pipermail/serviceability-dev/2015-March/016933.html > > The webrev for 8u-dev just adapts the copyright headers and the source > location (i.e. to before the "modular source change"). > > 8039173, which this change is a fix for on AIX already has been > integrated into 8u a while ago. > > Thanks, > Volker From ivan.gerasimov at oracle.com Wed Mar 18 13:53:28 2015 From: ivan.gerasimov at oracle.com (Ivan Gerasimov) Date: Wed, 18 Mar 2015 16:53:28 +0300 Subject: [8u-dev] Request for Approval to backport: 8072458: jdk/test/Makefile references (to be removed) win32 directory in jtreg Message-ID: <55098358.1080105@oracle.com> Hello! May I please get an approval to backport that change to the Makefile? Bug: https://bugs.openjdk.java.net/browse/JDK-8072458 Jdk9 change: http://hg.openjdk.java.net/jdk9/dev/jdk/rev/b8e8497c541c The fix from 9 applies cleanly. Sincerely yours, Ivan From sean.coffey at oracle.com Wed Mar 18 13:59:53 2015 From: sean.coffey at oracle.com (=?UTF-8?B?U2XDoW4gQ29mZmV5?=) Date: Wed, 18 Mar 2015 13:59:53 +0000 Subject: [8u-dev] Request for Approval to backport: 8072458: jdk/test/Makefile references (to be removed) win32 directory in jtreg In-Reply-To: <55098358.1080105@oracle.com> References: <55098358.1080105@oracle.com> Message-ID: <550984D9.7060103@oracle.com> Approved. Please add a suitable noreg- label to bug report. regards, Sean. On 18/03/15 13:53, Ivan Gerasimov wrote: > Hello! > > May I please get an approval to backport that change to the Makefile? > > Bug: https://bugs.openjdk.java.net/browse/JDK-8072458 > Jdk9 change: http://hg.openjdk.java.net/jdk9/dev/jdk/rev/b8e8497c541c > > The fix from 9 applies cleanly. > > Sincerely yours, > Ivan From david.dehaven at oracle.com Thu Mar 19 01:12:59 2015 From: david.dehaven at oracle.com (David DeHaven) Date: Wed, 18 Mar 2015 18:12:59 -0700 Subject: [8u60] Request for approval: 8075400: Cannot build hotspot in jdk8u on OSX 10.10 (Yosemite) Message-ID: <94953985-669E-467C-8949-BBEC18B305DB@oracle.com> JBS Issue: https://bugs.openjdk.java.net/browse/JDK-8075400 Webrev: http://cr.openjdk.java.net/~ddehaven/8075400/hotspot.0/ Discussion: http://mail.openjdk.java.net/pipermail/hotspot-dev/2015-March/017571.html -DrD- From david.holmes at oracle.com Thu Mar 19 01:58:50 2015 From: david.holmes at oracle.com (David Holmes) Date: Thu, 19 Mar 2015 11:58:50 +1000 Subject: [8u60] Request for approval: 8075400: Cannot build hotspot in jdk8u on OSX 10.10 (Yosemite) In-Reply-To: <94953985-669E-467C-8949-BBEC18B305DB@oracle.com> References: <94953985-669E-467C-8949-BBEC18B305DB@oracle.com> Message-ID: <550A2D5A.1070302@oracle.com> Hi David, Hotspot changes for 8u normally go in via the jdk8u/hs-dev forest and are then bulk integrated to jdk8u/jdk8u-dev after PIT, using a bulk approval request. So unless you propose to push this directly to jdk8u/jdk8u-dev this seperate approval request is not necessary. Cheers, David H. On 19/03/2015 11:12 AM, David DeHaven wrote: > > JBS Issue: https://bugs.openjdk.java.net/browse/JDK-8075400 > Webrev: http://cr.openjdk.java.net/~ddehaven/8075400/hotspot.0/ > Discussion: http://mail.openjdk.java.net/pipermail/hotspot-dev/2015-March/017571.html > > -DrD- > From david.dehaven at oracle.com Thu Mar 19 03:03:39 2015 From: david.dehaven at oracle.com (David DeHaven) Date: Wed, 18 Mar 2015 20:03:39 -0700 Subject: [8u60] Request for approval: 8075400: Cannot build hotspot in jdk8u on OSX 10.10 (Yosemite) In-Reply-To: <550A2D5A.1070302@oracle.com> References: <94953985-669E-467C-8949-BBEC18B305DB@oracle.com> <550A2D5A.1070302@oracle.com> Message-ID: Oh, right, thanks. I'll go ahead and submit the JPRT job then. -DrD- > Hi David, > > Hotspot changes for 8u normally go in via the jdk8u/hs-dev forest and are then bulk integrated to jdk8u/jdk8u-dev after PIT, using a bulk approval request. So unless you propose to push this directly to jdk8u/jdk8u-dev this seperate approval request is not necessary. > > Cheers, > David H. > > On 19/03/2015 11:12 AM, David DeHaven wrote: >> >> JBS Issue: https://bugs.openjdk.java.net/browse/JDK-8075400 >> Webrev: http://cr.openjdk.java.net/~ddehaven/8075400/hotspot.0/ >> Discussion: http://mail.openjdk.java.net/pipermail/hotspot-dev/2015-March/017571.html >> >> -DrD- >> From erik.joelsson at oracle.com Thu Mar 19 11:20:31 2015 From: erik.joelsson at oracle.com (Erik Joelsson) Date: Thu, 19 Mar 2015 12:20:31 +0100 Subject: [8u60] Request for approval: JDK-8075495: Update jtreg bin location in configure Message-ID: <550AB0FF.7010401@oracle.com> Hello, Could I please get an approval to backport this change from jdk9 to jdk8u? Bug: https://bugs.openjdk.java.net/browse/JDK-8075495 Review thread: http://mail.openjdk.java.net/pipermail/build-dev/2015-March/014608.html Jdk9 changeset: http://hg.openjdk.java.net/jdk9/dev/rev/5c624e587f07 The same patch applies to cleanly, except of course, for the generated configure script needing to be regenerated. /Erik From sean.coffey at oracle.com Thu Mar 19 11:24:01 2015 From: sean.coffey at oracle.com (=?UTF-8?B?U2XDoW4gQ29mZmV5?=) Date: Thu, 19 Mar 2015 11:24:01 +0000 Subject: [8u60] Request for approval: JDK-8075495: Update jtreg bin location in configure In-Reply-To: <550AB0FF.7010401@oracle.com> References: <550AB0FF.7010401@oracle.com> Message-ID: <550AB1D1.5060300@oracle.com> Approved. Might be good to add a suitable noreg- label to bug report. regards, Sean. On 19/03/2015 11:20, Erik Joelsson wrote: > Hello, > > Could I please get an approval to backport this change from jdk9 to > jdk8u? > > Bug: https://bugs.openjdk.java.net/browse/JDK-8075495 > Review thread: > http://mail.openjdk.java.net/pipermail/build-dev/2015-March/014608.html > Jdk9 changeset: http://hg.openjdk.java.net/jdk9/dev/rev/5c624e587f07 > > The same patch applies to cleanly, except of course, for the generated > configure script needing to be regenerated. > > /Erik From ivan.gerasimov at oracle.com Thu Mar 19 13:49:44 2015 From: ivan.gerasimov at oracle.com (Ivan Gerasimov) Date: Thu, 19 Mar 2015 16:49:44 +0300 Subject: [8u60] Request for Approval to Backport: 7065233: To interpret case-insensitive string locale independently Message-ID: <550AD3F8.7070309@oracle.com> Hello! Would you please approve backport of this? The patch applies cleanly. Was built/tested on all platforms, no failures. Bug: https://bugs.openjdk.java.net/browse/JDK-7065233 Jdk9 change: http://hg.openjdk.java.net/jdk9/dev/jdk/rev/2c1c21d11e58 Jdk9 review: http://mail.openjdk.java.net/pipermail/security-dev/2014-June/010664.html Sincerely yours, Ivan From sean.coffey at oracle.com Thu Mar 19 13:57:44 2015 From: sean.coffey at oracle.com (=?UTF-8?B?U2XDoW4gQ29mZmV5?=) Date: Thu, 19 Mar 2015 13:57:44 +0000 Subject: [8u60] Request for Approval to Backport: 7065233: To interpret case-insensitive string locale independently In-Reply-To: <550AD3F8.7070309@oracle.com> References: <550AD3F8.7070309@oracle.com> Message-ID: <550AD5D8.9040103@oracle.com> Approved. regards, Sean. On 19/03/2015 13:49, Ivan Gerasimov wrote: > Hello! > > Would you please approve backport of this? > The patch applies cleanly. > Was built/tested on all platforms, no failures. > > Bug: https://bugs.openjdk.java.net/browse/JDK-7065233 > Jdk9 change: http://hg.openjdk.java.net/jdk9/dev/jdk/rev/2c1c21d11e58 > Jdk9 review: > http://mail.openjdk.java.net/pipermail/security-dev/2014-June/010664.html > > Sincerely yours, > Ivan From ivan.gerasimov at oracle.com Thu Mar 19 15:43:15 2015 From: ivan.gerasimov at oracle.com (Ivan Gerasimov) Date: Thu, 19 Mar 2015 18:43:15 +0300 Subject: [8u60] Request for Approval to Backport: 8044860: Vectors and fixed length fields should be verified for allowed sizes. Message-ID: <550AEE93.8000107@oracle.com> Hello! Would you please approve backport of this fix? The unshuffled patch applies *almost* cleanly -- I only had to move the test file to the other directory. The tests run fine. Built/tested with JPRT on all platforms, no failures so far. Bug: https://bugs.openjdk.java.net/browse/JDK-8044860 Jdk9 change: http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/41b3e91dc2b2 Jdk9 review: http://mail.openjdk.java.net/pipermail/security-dev/2015-January/011636.html Sincerely yours, Ivan From sean.coffey at oracle.com Thu Mar 19 15:54:29 2015 From: sean.coffey at oracle.com (=?UTF-8?B?U2XDoW4gQ29mZmV5?=) Date: Thu, 19 Mar 2015 15:54:29 +0000 Subject: [8u60] Request for Approval to Backport: 8044860: Vectors and fixed length fields should be verified for allowed sizes. In-Reply-To: <550AEE93.8000107@oracle.com> References: <550AEE93.8000107@oracle.com> Message-ID: <550AF135.5090303@oracle.com> Approved. regards, Sean. On 19/03/2015 15:43, Ivan Gerasimov wrote: > Hello! > > Would you please approve backport of this fix? > The unshuffled patch applies *almost* cleanly -- I only had to move > the test file to the other directory. > The tests run fine. > Built/tested with JPRT on all platforms, no failures so far. > > Bug: https://bugs.openjdk.java.net/browse/JDK-8044860 > Jdk9 change: http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/41b3e91dc2b2 > Jdk9 review: > http://mail.openjdk.java.net/pipermail/security-dev/2015-January/011636.html > > Sincerely yours, > Ivan > From srikanth.adayapalam at oracle.com Fri Mar 20 10:57:52 2015 From: srikanth.adayapalam at oracle.com (Srikanth) Date: Fri, 20 Mar 2015 16:27:52 +0530 Subject: [8u60] Request for Approval to Backport: JDK-8069545: javac, shouldn't check nested stuck lambdas during overload resolution In-Reply-To: <550AD3F8.7070309@oracle.com> References: <550AD3F8.7070309@oracle.com> Message-ID: <550BFD30.1000600@oracle.com> Greetings, Please approve the backport this fix made for JDK9: JDK9 bug: https://bugs.openjdk.java.net/browse/JDK-8069545 JDK9 HG change log: http://hg.openjdk.java.net/jdk9/dev/langtools/rev/b93a15ac3bde JDK8 backport ticket: https://bugs.openjdk.java.net/browse/JDK-8069546 The same patch applies cleanly and langtools + compiler JCK tests are clean. TIA, Srikanth From sean.coffey at oracle.com Fri Mar 20 11:12:48 2015 From: sean.coffey at oracle.com (=?UTF-8?B?U2XDoW4gQ29mZmV5?=) Date: Fri, 20 Mar 2015 11:12:48 +0000 Subject: [8u60] Request for Approval to Backport: JDK-8069545: javac, shouldn't check nested stuck lambdas during overload resolution In-Reply-To: <550BFD30.1000600@oracle.com> References: <550AD3F8.7070309@oracle.com> <550BFD30.1000600@oracle.com> Message-ID: <550C00B0.3020605@oracle.com> Approved. regards, Sean. On 20/03/15 10:57, Srikanth wrote: > Greetings, > > Please approve the backport this fix made for JDK9: > > JDK9 bug: https://bugs.openjdk.java.net/browse/JDK-8069545 > JDK9 HG change log: > http://hg.openjdk.java.net/jdk9/dev/langtools/rev/b93a15ac3bde > JDK8 backport ticket: https://bugs.openjdk.java.net/browse/JDK-8069546 > > The same patch applies cleanly and langtools + compiler JCK tests are > clean. > > TIA, > Srikanth > > From sundararajan.athijegannathan at oracle.com Fri Mar 20 15:02:24 2015 From: sundararajan.athijegannathan at oracle.com (A. Sundararajan) Date: Fri, 20 Mar 2015 20:32:24 +0530 Subject: [8u60] approval request for 8075604: jjs exits even when non-daemon threads are still active Message-ID: <550C3680.4090400@oracle.com> Please approve. Bug: https://bugs.openjdk.java.net/browse/JDK-8075604 jdk9 review thread: http://mail.openjdk.java.net/pipermail/nashorn-dev/2015-March/004376.html jdk8u-dev webrev: http://cr.openjdk.java.net/~sundar/8075604/8u60/webrev.00/ Backported "as is" except for modular source layout difference. Thanks, -Sundar From sean.coffey at oracle.com Fri Mar 20 15:14:23 2015 From: sean.coffey at oracle.com (=?UTF-8?B?U2XDoW4gQ29mZmV5?=) Date: Fri, 20 Mar 2015 15:14:23 +0000 Subject: [8u60] approval request for 8075604: jjs exits even when non-daemon threads are still active In-Reply-To: <550C3680.4090400@oracle.com> References: <550C3680.4090400@oracle.com> Message-ID: <550C394F.6060007@oracle.com> Approved. regards, Sean. On 20/03/15 15:02, A. Sundararajan wrote: > Please approve. > > Bug: https://bugs.openjdk.java.net/browse/JDK-8075604 > jdk9 review thread: > http://mail.openjdk.java.net/pipermail/nashorn-dev/2015-March/004376.html > jdk8u-dev webrev: > http://cr.openjdk.java.net/~sundar/8075604/8u60/webrev.00/ > > Backported "as is" except for modular source layout difference. > > Thanks, > -Sundar From sean.coffey at oracle.com Fri Mar 20 15:19:20 2015 From: sean.coffey at oracle.com (=?UTF-8?B?U2XDoW4gQ29mZmV5?=) Date: Fri, 20 Mar 2015 15:19:20 +0000 Subject: [8u-dev] Request for approval : 8058547: Memory leak in ProtectionDomain cache Message-ID: <550C3A78.8070708@oracle.com> I'd like to backport this memory leak fix to jdk8u-dev. Patch applies cleanly post path shuffling. bug report : https://bugs.openjdk.java.net/browse/JDK-8058547 jdk 9 changeset : http://hg.openjdk.java.net/jdk9/dev/jdk/rev/57d07aa0a0eb regards, Sean. From rob.mckenna at oracle.com Fri Mar 20 16:29:06 2015 From: rob.mckenna at oracle.com (Rob McKenna) Date: Fri, 20 Mar 2015 16:29:06 +0000 Subject: [8u-dev] Request for approval : 8058547: Memory leak in ProtectionDomain cache In-Reply-To: <550C3A78.8070708@oracle.com> References: <550C3A78.8070708@oracle.com> Message-ID: <550C4AD2.80000@oracle.com> Approved. -Rob On 20/03/15 15:19, Se?n Coffey wrote: > I'd like to backport this memory leak fix to jdk8u-dev. Patch applies > cleanly post path shuffling. > > bug report : https://bugs.openjdk.java.net/browse/JDK-8058547 > jdk 9 changeset : > http://hg.openjdk.java.net/jdk9/dev/jdk/rev/57d07aa0a0eb > > regards, > Sean. From rob.mckenna at oracle.com Fri Mar 20 17:17:31 2015 From: rob.mckenna at oracle.com (Rob McKenna) Date: Fri, 20 Mar 2015 17:17:31 +0000 Subject: [8u-dev] Request for Approval - 8075039: (sctp) com/sun/nio/sctp/SctpMultiChannel/SendFailed.java fails on Solaris only Message-ID: <550C562B.7090406@oracle.com> Hi folks, Simple clean backport of a test fix: https://bugs.openjdk.java.net/browse/JDK-8075039 http://mail.openjdk.java.net/pipermail/net-dev/2015-March/008935.html http://hg.openjdk.java.net/jdk9/dev/jdk/rev/dd28613af6e4 -Rob From sean.coffey at oracle.com Fri Mar 20 17:24:14 2015 From: sean.coffey at oracle.com (=?UTF-8?B?U2XDoW4gQ29mZmV5?=) Date: Fri, 20 Mar 2015 17:24:14 +0000 Subject: [8u-dev] Request for Approval - 8075039: (sctp) com/sun/nio/sctp/SctpMultiChannel/SendFailed.java fails on Solaris only In-Reply-To: <550C562B.7090406@oracle.com> References: <550C562B.7090406@oracle.com> Message-ID: <550C57BE.1010802@oracle.com> Approved. regards, Sean. On 20/03/15 17:17, Rob McKenna wrote: > Hi folks, > > Simple clean backport of a test fix: > > https://bugs.openjdk.java.net/browse/JDK-8075039 > http://mail.openjdk.java.net/pipermail/net-dev/2015-March/008935.html > http://hg.openjdk.java.net/jdk9/dev/jdk/rev/dd28613af6e4 > > -Rob > From rob.mckenna at oracle.com Fri Mar 20 17:40:46 2015 From: rob.mckenna at oracle.com (Rob McKenna) Date: Fri, 20 Mar 2015 17:40:46 +0000 Subject: [8u-dev] Request for Review - 8072385: Only the first DNSName entry is checked for endpoint identification Message-ID: <550C5B9E.2090004@oracle.com> Hi folks, Looking for a review for the following: https://bugs.openjdk.java.net/browse/JDK-8072385 http://cr.openjdk.java.net/~robm/8072385/webrev.01/ -Rob From sean.coffey at oracle.com Fri Mar 20 17:53:12 2015 From: sean.coffey at oracle.com (=?UTF-8?B?U2XDoW4gQ29mZmV5?=) Date: Fri, 20 Mar 2015 17:53:12 +0000 Subject: [8u-dev] Request for approval for 8054037, 8055207 - Improve tracing for java.security.debug=certpath Message-ID: <550C5E88.2030009@oracle.com> Looking to backport these tracing enhancements to jdk8u-dev. 8054037: Improve tracing for java.security.debug=certpath 8055207: keystore and truststore debug output could be much better review thread : http://mail.openjdk.java.net/pipermail/security-dev/2015-February/011713.html jdk9 changeset : http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/f7352b8a3cbf The changes apply cleanly post path unshuffling (one minor copyright year line conflict) regards, Sean. From rob.mckenna at oracle.com Fri Mar 20 17:56:22 2015 From: rob.mckenna at oracle.com (Rob McKenna) Date: Fri, 20 Mar 2015 17:56:22 +0000 Subject: [8u-dev] Request for approval for 8054037, 8055207 - Improve tracing for java.security.debug=certpath In-Reply-To: <550C5E88.2030009@oracle.com> References: <550C5E88.2030009@oracle.com> Message-ID: <550C5F46.308@oracle.com> Approved. -Rob On 20/03/15 17:53, Se?n Coffey wrote: > Looking to backport these tracing enhancements to jdk8u-dev. > > 8054037: Improve tracing for java.security.debug=certpath > 8055207: keystore and truststore debug output could be much better > > review thread : > http://mail.openjdk.java.net/pipermail/security-dev/2015-February/011713.html > jdk9 changeset : > http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/f7352b8a3cbf > > The changes apply cleanly post path unshuffling (one minor copyright > year line conflict) > > regards, > Sean. From alejandro.murillo at oracle.com Fri Mar 20 18:38:43 2015 From: alejandro.murillo at oracle.com (Alejandro E Murillo) Date: Fri, 20 Mar 2015 12:38:43 -0600 Subject: [8u60] Request for approval for bulk integration of hs25.60-b08 Message-ID: <550C6933.1080901@oracle.com> Requesting approval to integrate hs25.60-b08 into jdk8u60-b08. A webrev is available at: http://cr.openjdk.java.net/~amurillo/8u60/hs25.60-b08-jdk8u60-b08.webrev/ Pre-integration testing is in progress; the integration will proceed only after SQE has analyzed the results and approved. The fixes in the proposed integration are below. All have undergone nightly testing and are already in a jdk9 repository. 8027914: Client JVM silently exit with fail exit code when running in compact(1,2) with options -Dcom.sun.management and -XX:+ManagementServer 8042796: jvmtiRedefineClasses.cpp: guarantee(false) failed: OLD and/or OBSOLETE method(s) found 8043770: File leak in MemNotifyThread::start() in hotspot.src.os.linux.vm.os_linux.cpp 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale 8074869: C2 code generator can replace -0.0f with +0.0f on Linux 8075144: new hotspot build - hs25.60-b08 8075400: Cannot build hotspot in jdk8u on OSX 10.10 (Yosemite) -- Alejandro From rob.mckenna at oracle.com Fri Mar 20 18:49:18 2015 From: rob.mckenna at oracle.com (Rob McKenna) Date: Fri, 20 Mar 2015 18:49:18 +0000 Subject: [8u60] Request for approval for bulk integration of hs25.60-b08 In-Reply-To: <550C6933.1080901@oracle.com> References: <550C6933.1080901@oracle.com> Message-ID: <550C6BAE.8030806@oracle.com> Approved. -Rob On 20/03/15 18:38, Alejandro E Murillo wrote: > Requesting approval to integrate hs25.60-b08 into jdk8u60-b08. > > A webrev is available at: > > http://cr.openjdk.java.net/~amurillo/8u60/hs25.60-b08-jdk8u60-b08.webrev/ > > Pre-integration testing is in progress; the integration will proceed > only after SQE has analyzed the results and approved. > > The fixes in the proposed integration are below. All have undergone > nightly testing and are already in a jdk9 repository. > > 8027914: Client JVM silently exit with fail exit code when running in > compact(1,2) with options -Dcom.sun.management and -XX:+ManagementServer > 8042796: jvmtiRedefineClasses.cpp: guarantee(false) failed: OLD and/or > OBSOLETE method(s) found > 8043770: File leak in MemNotifyThread::start() in > hotspot.src.os.linux.vm.os_linux.cpp > 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in > RedefineClasses does not scale > 8074869: C2 code generator can replace -0.0f with +0.0f on Linux > 8075144: new hotspot build - hs25.60-b08 > 8075400: Cannot build hotspot in jdk8u on OSX 10.10 (Yosemite) > From Xuelei.Fan at Oracle.COM Sat Mar 21 02:30:44 2015 From: Xuelei.Fan at Oracle.COM (Xuelei Fan) Date: Sat, 21 Mar 2015 10:30:44 +0800 Subject: [8u-dev] Request for Review - 8072385: Only the first DNSName entry is checked for endpoint identification In-Reply-To: <550C5B9E.2090004@oracle.com> References: <550C5B9E.2090004@oracle.com> Message-ID: <550CD7D4.1040307@Oracle.COM> Looks fine to me. Xuelei On 3/21/2015 1:40 AM, Rob McKenna wrote: > Hi folks, > > Looking for a review for the following: > > https://bugs.openjdk.java.net/browse/JDK-8072385 > http://cr.openjdk.java.net/~robm/8072385/webrev.01/ > > -Rob > From bitterfoxc at gmail.com Sat Mar 21 08:53:01 2015 From: bitterfoxc at gmail.com (ShinyaYoshida) Date: Sat, 21 Mar 2015 17:53:01 +0900 Subject: RFR 8075664: NPE for illegall super call with MethodReference Message-ID: Hi, compier-dev, jdk8u-dev, I found a bug and have a fix for the bug. Please review my patch. Bugs: https://bugs.openjdk.java.net/browse/JDK-8075664 Webrev: http://cr.openjdk.java.net/~shinyafox/8075664/webrev.00/ Regards, shinyafox(Shinya Yoshida) From andrew.brygin at oracle.com Mon Mar 23 08:27:39 2015 From: andrew.brygin at oracle.com (Andrew Brygin) Date: Mon, 23 Mar 2015 11:27:39 +0300 Subject: [8u60] Request for approval: 8074954: ImageInputStreamImpl.readShort/readInt do not behave correctly at EOF Message-ID: <550FCE7B.5060406@oracle.com> Hello, could you please approve a direct backport to 8u-dev? Bug: https://bugs.openjdk.java.net/browse/JDK-8074954 Fix: http://hg.openjdk.java.net/jdk9/client/jdk/rev/c08405d53154 Review thread: http://mail.openjdk.java.net/pipermail/2d-dev/2015-March/005199.html Thanks, Andrew From sean.coffey at oracle.com Mon Mar 23 08:31:30 2015 From: sean.coffey at oracle.com (=?windows-1252?Q?Se=E1n_Coffey?=) Date: Mon, 23 Mar 2015 08:31:30 +0000 Subject: [8u60] Request for approval: 8074954: ImageInputStreamImpl.readShort/readInt do not behave correctly at EOF In-Reply-To: <550FCE7B.5060406@oracle.com> References: <550FCE7B.5060406@oracle.com> Message-ID: <550FCF62.20907@oracle.com> Approved. regards, Sean. On 23/03/2015 08:27, Andrew Brygin wrote: > Hello, > > could you please approve a direct backport to 8u-dev? > > Bug: https://bugs.openjdk.java.net/browse/JDK-8074954 > Fix: http://hg.openjdk.java.net/jdk9/client/jdk/rev/c08405d53154 > Review thread: > http://mail.openjdk.java.net/pipermail/2d-dev/2015-March/005199.html > > Thanks, > Andrew From rob.mckenna at oracle.com Mon Mar 23 15:21:25 2015 From: rob.mckenna at oracle.com (Rob McKenna) Date: Mon, 23 Mar 2015 15:21:25 +0000 Subject: [8u-dev] Request for Approval - 8072385: Only the first DNSName entry is checked for endpoint identification Message-ID: <55102F75.4040307@oracle.com> Hi folks, Requesting approval for this change: bug: https://bugs.openjdk.java.net/browse/JDK-8072385 webrev: http://cr.openjdk.java.net/~robm/8072385/webrev.01/ review: http://mail.openjdk.java.net/pipermail/jdk8u-dev/2015-March/003268.html -Rob From michail.chernov at oracle.com Mon Mar 23 15:33:42 2015 From: michail.chernov at oracle.com (Michail Chernov) Date: Mon, 23 Mar 2015 18:33:42 +0300 Subject: [8u60] Request for backport approval: 8055416 Several vm/gc/heap/summary "After GC" events emitted for the same GC ID Message-ID: <55103256.9050501@oracle.com> Dear colleagues, I'd like to get approval to backport changes from JDK9 to 8u. Webrev: http://cr.openjdk.java.net/~eistepan/~mchernov/8055416/webrev.00/ Original issue was fixed and approved at August 2014. Here is the mail thread: http://mail.openjdk.java.net/pipermail/hotspot-gc-dev/2014-August/010549.html JDK9 webrev: http://cr.openjdk.java.net/~stefank/8055416/webrev.00/ Original bug is https://bugs.openjdk.java.net/browse/JDK-8055416 " For client VM and G1 GC several vm/gc/heap/summary events with property when='After GC' could be emitted for the same GC ID. At the same time there will be only one 'Before GC' vm/gc/heap/summary event for the same GC ID." Testing: JDK with changes was built locally. Closed JFR test has passed. Thanks, Michail From sean.coffey at oracle.com Mon Mar 23 15:52:37 2015 From: sean.coffey at oracle.com (=?UTF-8?B?U2XDoW4gQ29mZmV5?=) Date: Mon, 23 Mar 2015 15:52:37 +0000 Subject: [8u-dev] Request for Approval - 8072385: Only the first DNSName entry is checked for endpoint identification In-Reply-To: <55102F75.4040307@oracle.com> References: <55102F75.4040307@oracle.com> Message-ID: <551036C5.2020105@oracle.com> Approved. regards, Sean. On 23/03/15 15:21, Rob McKenna wrote: > Hi folks, > > Requesting approval for this change: > > bug: https://bugs.openjdk.java.net/browse/JDK-8072385 > webrev: http://cr.openjdk.java.net/~robm/8072385/webrev.01/ > review: > http://mail.openjdk.java.net/pipermail/jdk8u-dev/2015-March/003268.html > > -Rob From sean.coffey at oracle.com Mon Mar 23 15:56:02 2015 From: sean.coffey at oracle.com (=?UTF-8?B?U2XDoW4gQ29mZmV5?=) Date: Mon, 23 Mar 2015 15:56:02 +0000 Subject: [8u60] Request for backport approval: 8055416 Several vm/gc/heap/summary "After GC" events emitted for the same GC ID In-Reply-To: <55103256.9050501@oracle.com> References: <55103256.9050501@oracle.com> Message-ID: <55103792.5020508@oracle.com> Hi Michail, this is a hotspot related fix and should enter 8u codeline via the hotspot team forest : http://hg.openjdk.java.net/jdk8u/hs-dev/ No approvals are necessary there as the gatekeeper will submit a bulk approval integration request. regards, Sean. On 23/03/15 15:33, Michail Chernov wrote: > Dear colleagues, > > I'd like to get approval to backport changes from JDK9 to 8u. > > Webrev: http://cr.openjdk.java.net/~eistepan/~mchernov/8055416/webrev.00/ > > > Original issue was fixed and approved at August 2014. Here is the mail > thread: > http://mail.openjdk.java.net/pipermail/hotspot-gc-dev/2014-August/010549.html > JDK9 webrev: http://cr.openjdk.java.net/~stefank/8055416/webrev.00/ > > Original bug is https://bugs.openjdk.java.net/browse/JDK-8055416 > " > > For client VM and G1 GC several vm/gc/heap/summary events with property > when='After GC' could be emitted for the same GC ID. > At the same time there will be only one 'Before GC' vm/gc/heap/summary > event for the same GC ID." > > > > Testing: JDK with changes was built locally. Closed JFR test has passed. > > Thanks, > Michail From lana.steuck at oracle.com Mon Mar 23 21:24:38 2015 From: lana.steuck at oracle.com (lana.steuck at oracle.com) Date: Mon, 23 Mar 2015 14:24:38 -0700 (PDT) Subject: jdk8u-b08: jdk8u-dev Message-ID: <201503232124.t2NLOc9j005473@sc11152554.us.oracle.com> http://hg.openjdk.java.net/jdk8u/jdk8u/rev/0785e45b19c8 http://hg.openjdk.java.net/jdk8u/jdk8u/nashorn/rev/e024db176497 http://hg.openjdk.java.net/jdk8u/jdk8u/langtools/rev/e5b93c508212 http://hg.openjdk.java.net/jdk8u/jdk8u/jdk/rev/104c06bb638c http://hg.openjdk.java.net/jdk8u/jdk8u/jaxws/rev/078fde829e87 http://hg.openjdk.java.net/jdk8u/jdk8u/jaxp/rev/0ef3e20f1d48 http://hg.openjdk.java.net/jdk8u/jdk8u/hotspot/rev/5755b2aee8e8 http://hg.openjdk.java.net/jdk8u/jdk8u/corba/rev/b184ceca742e --- All the fixes will be tested during promotion (no PIT testing at this point): List of all fixes: =================== JDK-8074668 client-libs [macosx] Mac 10.10: Application run with splash screen has focus issue JDK-8048289 client-libs Gtk: call to UIManager.getSystemLookAndFeelClassName() leads to crash JDK-8072900 client-libs Mouse events are captured by the wrong menu in OS X JDK-8073008 client-libs press-and-hold input method for accented characters works incorrectly JDK-8074687 core-libs Add tests for JSON parsing of numeric keys JDK-8072426 core-libs Can't compare Java enums to strings JDK-8074031 core-libs Canonicalize "is a JS string" tests JDK-8066217 core-libs Fuzzing bug: new ArrayBuffer() throws exception JDK-8035712 core-libs Investigate if RuntimeCallSite linkage can be removed JDK-8072458 core-libs jdk/test/Makefile references (to be removed) win32 directory in jtreg JDK-8074694 core-libs Lazy conversion of ZipEntry time JDK-8074791 core-libs Long-form date format incorrect month string for Finnish locale JDK-8074484 core-libs More aggressive value discarding JDK-8062030 core-libs Nashorn bug retrieving array property after key string concatenation JDK-8074487 core-libs Static analysis of IfNode should consider terminating branches JDK-8073148 core-svc "The server has decided to close this client connection" repeated con JDK-8071687 core-svc AIX port of "8039173: Propagate errors from Diagnostic Commands as exc JDK-8071657 core-svc JDI ObjectReferenceImpl.invokeMethod() validation fails for virtual in JDK-8074564 deploy Replace missing APIs from 8u40 JDK-8053902 hotspot Fix for 8030115 breaks build on Windows and Solaris JDK-8025636 hotspot Hide lambda proxy frames in stacktraces JDK-8068927 infrastructure AARCH64: better handling of aarch64- triples JDK-8064357 infrastructure AARCH64: Top-level JDK changes JDK-8074554 infrastructure Create custom hook for running after AC_OUTPUT JDK-8069057 infrastructure Make sure configure is run by bash JDK-8072732 infrastructure Regression in configure due to JDK-8069057 JDK-8075495 infrastructure Update jtreg bin location in configure JDK-7065233 security-libs To interpret case-insensitive string locale independently JDK-8072461 tools Table's field width in "Use" page generated by javadoc with '-s' is un From sundararajan.athijegannathan at oracle.com Tue Mar 24 09:11:22 2015 From: sundararajan.athijegannathan at oracle.com (A. Sundararajan) Date: Tue, 24 Mar 2015 14:41:22 +0530 Subject: [8u60] approval request for 8074410: Startup time: Port shell.js to Java Message-ID: <55112A3A.1000808@oracle.com> Please approve. Bug: https://bugs.openjdk.java.net/browse/JDK-8074410 jdk9 review thread: http://mail.openjdk.java.net/pipermail/nashorn-dev/2015-March/004387.html jdk8u-dev webrev: http://cr.openjdk.java.net/~sundar/8074410/8u60/ Backported "as is" except for modular source layout difference. Thanks, -Sundar From sean.coffey at oracle.com Tue Mar 24 09:13:45 2015 From: sean.coffey at oracle.com (=?UTF-8?B?U2XDoW4gQ29mZmV5?=) Date: Tue, 24 Mar 2015 09:13:45 +0000 Subject: [8u60] approval request for 8074410: Startup time: Port shell.js to Java In-Reply-To: <55112A3A.1000808@oracle.com> References: <55112A3A.1000808@oracle.com> Message-ID: <55112AC9.4000008@oracle.com> Approved. regards, Sean. On 24/03/2015 09:11, A. Sundararajan wrote: > Please approve. > > Bug: https://bugs.openjdk.java.net/browse/JDK-8074410 > jdk9 review thread: > http://mail.openjdk.java.net/pipermail/nashorn-dev/2015-March/004387.html > jdk8u-dev webrev: http://cr.openjdk.java.net/~sundar/8074410/8u60/ > > Backported "as is" except for modular source layout difference. > > Thanks, > -Sundar From alejandro.murillo at oracle.com Tue Mar 24 15:31:37 2015 From: alejandro.murillo at oracle.com (Alejandro E Murillo) Date: Tue, 24 Mar 2015 09:31:37 -0600 Subject: jdk8u60-b08: HotSpot Message-ID: <55118359.7080309@oracle.com> hs25.60-b08 has been integrated into jdk8u60-b08. http://hg.openjdk.java.net/jdk8u/jdk8u/rev/0785e45b19c8 http://hg.openjdk.java.net/jdk8u/jdk8u/corba/rev/b184ceca742e http://hg.openjdk.java.net/jdk8u/jdk8u/hotspot/rev/bf68e15dc8fe http://hg.openjdk.java.net/jdk8u/jdk8u/jaxp/rev/0ef3e20f1d48 http://hg.openjdk.java.net/jdk8u/jdk8u/jaxws/rev/078fde829e87 http://hg.openjdk.java.net/jdk8u/jdk8u/jdk/rev/478602cc17e2 http://hg.openjdk.java.net/jdk8u/jdk8u/langtools/rev/e5b93c508212 http://hg.openjdk.java.net/jdk8u/jdk8u/nashorn/rev/e024db176497 Component : VM Status : 0 major failures, 1 minor failure Date : 03/24/2015 at 17:00 MSK Tested By : STT_VM Cost(total man-days): 1 Workspace : 2015-03-20-161314.amurillo.hs25-60-b08-snapshot Bundles : 2015-03-20-161314.amurillo.hs25-60-b08-snapshot Platforms : Others Tests : Log : link Browsers : NA Patches : NA Number of Tests Executed : 362110 passed tests, 3253 failed tests (1 new failure) Bug verification status: ====================================== Tested, Pass: Tested, Pass (partial fixes): Tested, Fail: Untested bug fixes: 8027914: Client JVM silently exit with fail exit code when running in compact(1,2) with options -Dcom.sun.management and -XX:+ManagementServer 8042796: jvmtiRedefineClasses.cpp: guarantee(false) failed: OLD and/or OBSOLETE method(s) found 8043770: File leak in MemNotifyThread::start() in hotspot.src.os.linux.vm.os_linux.cpp 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale 8074869: C2 code generator can replace -0.0f with +0.0f on Linux 8075144: new hotspot build - hs25.60-b08 8075400: Cannot build hotspot in jdk8u on OSX 10.10 (Yosemite) New bugs filed: Bugs in PIT build: 8075805: SIGSEGV in GuardedMemory::print_on Bugs in earlier promoted build: Number of PIT requested: 1 Integration target J2SE build number: jdk8u60-b08 Issues and Notes: This is PIT for HS25.60-b08 for jdk8u60-b08. Go for integration. -- Alejandro From sundararajan.athijegannathan at oracle.com Wed Mar 25 09:37:45 2015 From: sundararajan.athijegannathan at oracle.com (A. Sundararajan) Date: Wed, 25 Mar 2015 15:07:45 +0530 Subject: [8u60] 8012190: Global scope should be initialized lazily Message-ID: <551281E9.2030305@oracle.com> Please approve. Bug: https://bugs.openjdk.java.net/browse/JDK-8012190 jdk9 review thread: http://mail.openjdk.java.net/pipermail/nashorn-dev/2015-March/004392.html jdk8u-dev webrev: http://cr.openjdk.java.net/~sundar/8012190/8u60/webrev.00/ Backported "as is" except for modular source layout change. Thanks, -Sundar From sean.coffey at oracle.com Wed Mar 25 10:00:02 2015 From: sean.coffey at oracle.com (=?UTF-8?B?U2XDoW4gQ29mZmV5?=) Date: Wed, 25 Mar 2015 10:00:02 +0000 Subject: [8u60] 8012190: Global scope should be initialized lazily In-Reply-To: <551281E9.2030305@oracle.com> References: <551281E9.2030305@oracle.com> Message-ID: <55128722.3050809@oracle.com> Approved. regards, Sean. On 25/03/2015 09:37, A. Sundararajan wrote: > Please approve. > > Bug: https://bugs.openjdk.java.net/browse/JDK-8012190 > jdk9 review thread: > http://mail.openjdk.java.net/pipermail/nashorn-dev/2015-March/004392.html > jdk8u-dev webrev: > http://cr.openjdk.java.net/~sundar/8012190/8u60/webrev.00/ > > Backported "as is" except for modular source layout change. > > Thanks, > -Sundar > From volker.simonis at gmail.com Wed Mar 25 17:48:24 2015 From: volker.simonis at gmail.com (Volker Simonis) Date: Wed, 25 Mar 2015 18:48:24 +0100 Subject: [8u60] request for approval: "8067923: AIX: link libjvm.so with -bernotok..." and "8075858: AIX: clean-up HotSpot make files" Message-ID: Hi, could you please approve and sponsor the backport of the following two, AIX-only HotSpot fixes to jd8u-dev: 8067923: AIX: link libjvm.so with -bernotok to detect missing symbols at build time and suppress warning 1540-1639 Bug: https://bugs.openjdk.java.net/browse/JDK-8067923 Webrev: http://cr.openjdk.java.net/~simonis/webrevs/8067923/ Review: http://mail.openjdk.java.net/pipermail/hotspot-dev/2014-December/016568.html URL: http://hg.openjdk.java.net/jdk9/hs-rt/hotspot/rev/c0f8994d6ee6 8075858: AIX: clean-up HotSpot make files Bug: https://bugs.openjdk.java.net/browse/JDK-8075858 Webrev: http://cr.openjdk.java.net/~simonis/webrevs/2015/8075858/ Review: http://mail.openjdk.java.net/pipermail/hotspot-dev/2015-March/017659.html URL: http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/rev/3891403eb18e Both changes cleanly apply to the current 8u-dev/hotspot repository and only touch AIX-specific make files. Thank you and best regards, Volker From vladimir.kozlov at oracle.com Wed Mar 25 21:00:37 2015 From: vladimir.kozlov at oracle.com (Vladimir Kozlov) Date: Wed, 25 Mar 2015 14:00:37 -0700 Subject: [8u60] request for approval: "8067923: AIX: link libjvm.so with -bernotok..." and "8075858: AIX: clean-up HotSpot make files" In-Reply-To: References: Message-ID: <551321F5.7020800@oracle.com> Hi, Since it is only hotspot changes they will go through jdk8u/hs-dev/hotspot. Alejandro will merge them to 8u after testing. No need for separate approval from jdk8u. Next time, please, ask for Hotspot changes backport on hotspot-dev mailing list. I'm sponsoring backport - it is in JPRT queue. Thanks, Vladimir On 3/25/15 10:48 AM, Volker Simonis wrote: > Hi, > > could you please approve and sponsor the backport of the following > two, AIX-only HotSpot fixes to jd8u-dev: > > 8067923: AIX: link libjvm.so with -bernotok to detect missing symbols > at build time and suppress warning 1540-1639 > > Bug: https://bugs.openjdk.java.net/browse/JDK-8067923 > Webrev: http://cr.openjdk.java.net/~simonis/webrevs/8067923/ > Review: http://mail.openjdk.java.net/pipermail/hotspot-dev/2014-December/016568.html > URL: http://hg.openjdk.java.net/jdk9/hs-rt/hotspot/rev/c0f8994d6ee6 > > > 8075858: AIX: clean-up HotSpot make files > > Bug: https://bugs.openjdk.java.net/browse/JDK-8075858 > Webrev: http://cr.openjdk.java.net/~simonis/webrevs/2015/8075858/ > Review: http://mail.openjdk.java.net/pipermail/hotspot-dev/2015-March/017659.html > URL: http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/rev/3891403eb18e > > Both changes cleanly apply to the current 8u-dev/hotspot repository > and only touch AIX-specific make files. > > Thank you and best regards, > Volker > From sean.coffey at oracle.com Wed Mar 25 23:05:38 2015 From: sean.coffey at oracle.com (=?UTF-8?B?U2XDoW4gQ29mZmV5?=) Date: Wed, 25 Mar 2015 23:05:38 +0000 Subject: [8u60] request for approval: "8067923: AIX: link libjvm.so with -bernotok..." and "8075858: AIX: clean-up HotSpot make files" In-Reply-To: <551321F5.7020800@oracle.com> References: <551321F5.7020800@oracle.com> Message-ID: <55133F42.8090100@oracle.com> Thanks Vladimir! regards, Sean. On 25/03/2015 21:00, Vladimir Kozlov wrote: > Hi, > > Since it is only hotspot changes they will go through > jdk8u/hs-dev/hotspot. Alejandro will merge them to 8u after testing. > No need for separate approval from jdk8u. > > Next time, please, ask for Hotspot changes backport on hotspot-dev > mailing list. > > I'm sponsoring backport - it is in JPRT queue. > > Thanks, > Vladimir > > On 3/25/15 10:48 AM, Volker Simonis wrote: >> Hi, >> >> could you please approve and sponsor the backport of the following >> two, AIX-only HotSpot fixes to jd8u-dev: >> >> 8067923: AIX: link libjvm.so with -bernotok to detect missing symbols >> at build time and suppress warning 1540-1639 >> >> Bug: https://bugs.openjdk.java.net/browse/JDK-8067923 >> Webrev: http://cr.openjdk.java.net/~simonis/webrevs/8067923/ >> Review: >> http://mail.openjdk.java.net/pipermail/hotspot-dev/2014-December/016568.html >> URL: http://hg.openjdk.java.net/jdk9/hs-rt/hotspot/rev/c0f8994d6ee6 >> >> >> 8075858: AIX: clean-up HotSpot make files >> >> Bug: https://bugs.openjdk.java.net/browse/JDK-8075858 >> Webrev: http://cr.openjdk.java.net/~simonis/webrevs/2015/8075858/ >> Review: >> http://mail.openjdk.java.net/pipermail/hotspot-dev/2015-March/017659.html >> URL: http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/rev/3891403eb18e >> >> Both changes cleanly apply to the current 8u-dev/hotspot repository >> and only touch AIX-specific make files. >> >> Thank you and best regards, >> Volker >> From masayoshi.okutsu at oracle.com Thu Mar 26 05:46:20 2015 From: masayoshi.okutsu at oracle.com (Masayoshi Okutsu) Date: Thu, 26 Mar 2015 14:46:20 +0900 Subject: [8u60] Request for approval: 8075173: DateFormat in german locale returns wrong value for month march Message-ID: <55139D2C.6080101@oracle.com> Hello, Could you please approve backport of the fix for 8075173 to jdk8u-dev? The fix is identical to the JDK 9 one. Bug: https://bugs.openjdk.java.net/browse/JDK-8075173 Webrev: http://cr.openjdk.java.net/~okutsu/8u/8075173/webrev.00/ Review thread: http://mail.openjdk.java.net/pipermail/i18n-dev/2015-March/001644.html JDK9 changeset: http://hg.openjdk.java.net/jdk9/dev/jdk/rev/a379fd7e28b9 Thanks, Masayoshi From sean.coffey at oracle.com Thu Mar 26 08:13:42 2015 From: sean.coffey at oracle.com (=?ISO-8859-1?Q?Se=E1n_Coffey?=) Date: Thu, 26 Mar 2015 08:13:42 +0000 Subject: [8u60] Request for approval: 8075173: DateFormat in german locale returns wrong value for month march In-Reply-To: <55139D2C.6080101@oracle.com> References: <55139D2C.6080101@oracle.com> Message-ID: <644555E5-22D0-4B68-AB2A-D4E216548025@oracle.com> Approved. Regards, Sean. On 26 March 2015 05:46:20 GMT+00:00, Masayoshi Okutsu wrote: >Hello, > >Could you please approve backport of the fix for 8075173 to jdk8u-dev? >The fix is identical to the JDK 9 one. > >Bug: >https://bugs.openjdk.java.net/browse/JDK-8075173 >Webrev: >http://cr.openjdk.java.net/~okutsu/8u/8075173/webrev.00/ >Review thread: >http://mail.openjdk.java.net/pipermail/i18n-dev/2015-March/001644.html >JDK9 changeset: >http://hg.openjdk.java.net/jdk9/dev/jdk/rev/a379fd7e28b9 > >Thanks, >Masayoshi From volker.simonis at gmail.com Thu Mar 26 08:27:51 2015 From: volker.simonis at gmail.com (Volker Simonis) Date: Thu, 26 Mar 2015 09:27:51 +0100 Subject: [8u60] request for approval: "8067923: AIX: link libjvm.so with -bernotok..." and "8075858: AIX: clean-up HotSpot make files" In-Reply-To: <551321F5.7020800@oracle.com> References: <551321F5.7020800@oracle.com> Message-ID: On Wed, Mar 25, 2015 at 10:00 PM, Vladimir Kozlov wrote: > Hi, > > Since it is only hotspot changes they will go through jdk8u/hs-dev/hotspot. > Alejandro will merge them to 8u after testing. > No need for separate approval from jdk8u. > > Next time, please, ask for Hotspot changes backport on hotspot-dev mailing > list. > Sorry, I missed that. > I'm sponsoring backport - it is in JPRT queue. > Thanks a lot, Vladimir! Regards, Volker > Thanks, > Vladimir > > > On 3/25/15 10:48 AM, Volker Simonis wrote: >> >> Hi, >> >> could you please approve and sponsor the backport of the following >> two, AIX-only HotSpot fixes to jd8u-dev: >> >> 8067923: AIX: link libjvm.so with -bernotok to detect missing symbols >> at build time and suppress warning 1540-1639 >> >> Bug: https://bugs.openjdk.java.net/browse/JDK-8067923 >> Webrev: http://cr.openjdk.java.net/~simonis/webrevs/8067923/ >> Review: >> http://mail.openjdk.java.net/pipermail/hotspot-dev/2014-December/016568.html >> URL: http://hg.openjdk.java.net/jdk9/hs-rt/hotspot/rev/c0f8994d6ee6 >> >> >> 8075858: AIX: clean-up HotSpot make files >> >> Bug: https://bugs.openjdk.java.net/browse/JDK-8075858 >> Webrev: http://cr.openjdk.java.net/~simonis/webrevs/2015/8075858/ >> Review: >> http://mail.openjdk.java.net/pipermail/hotspot-dev/2015-March/017659.html >> URL: http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/rev/3891403eb18e >> >> Both changes cleanly apply to the current 8u-dev/hotspot repository >> and only touch AIX-specific make files. >> >> Thank you and best regards, >> Volker >> > From hannes.wallnoefer at oracle.com Thu Mar 26 14:04:24 2015 From: hannes.wallnoefer at oracle.com (Hannes Wallnoefer) Date: Thu, 26 Mar 2015 15:04:24 +0100 Subject: [8u60] Request for approval for JDK-8075006, JDK-8075927, and JDK-8073868 Message-ID: <551411E8.4040405@oracle.com> Please approve backport of the following three Nashorn bugs to 8u60: Bug: https://bugs.openjdk.java.net/browse/JDK-8075006 Webrev: http://cr.openjdk.java.net/~hannesw/8075006/ Review thread: http://mail.openjdk.java.net/pipermail/nashorn-dev/2015-March/004335.html Bug: https://bugs.openjdk.java.net/browse/JDK-8075927 Webrev: http://cr.openjdk.java.net/~hannesw/8075927/ Review thread: http://mail.openjdk.java.net/pipermail/nashorn-dev/2015-March/004399.html Bug: https://bugs.openjdk.java.net/browse/JDK-8073868 Webrev: http://cr.openjdk.java.net/~hannesw/8073868/ Review thread: http://mail.openjdk.java.net/pipermail/nashorn-dev/2015-March/004397.html The changes apply cleanly to 8u-dev after reshuffling path names. Thanks, Hannes From rob.mckenna at oracle.com Thu Mar 26 15:15:09 2015 From: rob.mckenna at oracle.com (Rob McKenna) Date: Thu, 26 Mar 2015 15:15:09 +0000 Subject: [8u60] Request for approval for JDK-8075006, JDK-8075927, and JDK-8073868 In-Reply-To: <551411E8.4040405@oracle.com> References: <551411E8.4040405@oracle.com> Message-ID: <5514227D.5000300@oracle.com> Approved. -Rob On 26/03/15 14:04, Hannes Wallnoefer wrote: > Please approve backport of the following three Nashorn bugs to 8u60: > > Bug: https://bugs.openjdk.java.net/browse/JDK-8075006 > Webrev: http://cr.openjdk.java.net/~hannesw/8075006/ > Review thread: > http://mail.openjdk.java.net/pipermail/nashorn-dev/2015-March/004335.html > > Bug: https://bugs.openjdk.java.net/browse/JDK-8075927 > Webrev: http://cr.openjdk.java.net/~hannesw/8075927/ > Review thread: > http://mail.openjdk.java.net/pipermail/nashorn-dev/2015-March/004399.html > > Bug: https://bugs.openjdk.java.net/browse/JDK-8073868 > Webrev: http://cr.openjdk.java.net/~hannesw/8073868/ > Review thread: > http://mail.openjdk.java.net/pipermail/nashorn-dev/2015-March/004397.html > > The changes apply cleanly to 8u-dev after reshuffling path names. > > Thanks, > Hannes From sean.coffey at oracle.com Thu Mar 26 15:47:20 2015 From: sean.coffey at oracle.com (=?UTF-8?B?U2XDoW4gQ29mZmV5?=) Date: Thu, 26 Mar 2015 15:47:20 +0000 Subject: [8u communication] - Removal of jdk8u40-dev forest Message-ID: <55142A08.5010402@oracle.com> To help keep the mercurial server clean of old unnecessary development forests, I'd like to suggest that the jdk8u/jdk8u40-dev forest be deleted. If I hear no objections before COB tomorrow (Friday), I'll ask ops to delete that forest. regards, Sean. From alexander.zvegintsev at oracle.com Thu Mar 26 17:06:46 2015 From: alexander.zvegintsev at oracle.com (Alexander Zvegintsev) Date: Thu, 26 Mar 2015 20:06:46 +0300 Subject: [8u60] Request for approval 8072088: [PIT] NPE in DnD tests apparently because of the fix to JDK-8061636 Message-ID: <55143CA6.7000207@oracle.com> Hello, This is a request for approval to backport a fix from the jdk9 to the jdk8u-dev repository. Changes applies cleanly to jdk8u-dev after path reshuffling. The bug: https://bugs.openjdk.java.net/browse/JDK-8072088 The fix: http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/d96e70a49103 The review: http://mail.openjdk.java.net/pipermail/awt-dev/2015-February/008903.html -- Thanks, Alexander. From alexander.zvegintsev at oracle.com Thu Mar 26 17:16:17 2015 From: alexander.zvegintsev at oracle.com (Alexander Zvegintsev) Date: Thu, 26 Mar 2015 20:16:17 +0300 Subject: [8u60] Request for approval 8056151 and 8074921 Message-ID: <55143EE1.9020201@oracle.com> Hello, This is a request for approval to backport a couple of fixes from the jdk9 to the jdk8u-dev repository. Changes applies cleanly to jdk8u-dev after path reshuffling. Switching to GTK L&F on-the-fly leads to X Window System error RenderBadPicture The bug: https://bugs.openjdk.java.net/browse/JDK-8056151 The fix: http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/b7739fc3663f The review: http://mail.openjdk.java.net/pipermail/awt-dev/2015-March/008974.html OS X build broken by reference to XToolkit The bug: https://bugs.openjdk.java.net/browse/JDK-8074921 The fix: http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/c8156e81be2f The review: http://mail.openjdk.java.net/pipermail/awt-dev/2015-March/009022.html -- Thanks, Alexander. From rob.mckenna at oracle.com Thu Mar 26 17:27:53 2015 From: rob.mckenna at oracle.com (Rob McKenna) Date: Thu, 26 Mar 2015 17:27:53 +0000 Subject: [8u60] Request for approval 8056151 and 8074921 In-Reply-To: <55143EE1.9020201@oracle.com> References: <55143EE1.9020201@oracle.com> Message-ID: <55144199.6010805@oracle.com> Approved. -Rob On 26/03/15 17:16, Alexander Zvegintsev wrote: > Hello, > > This is a request for approval to backport a couple of fixes from the > jdk9 to the jdk8u-dev repository. > Changes applies cleanly to jdk8u-dev after path reshuffling. > > > Switching to GTK L&F on-the-fly leads to X Window System error > RenderBadPicture > > The bug: https://bugs.openjdk.java.net/browse/JDK-8056151 > The fix: http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/b7739fc3663f > The review: > http://mail.openjdk.java.net/pipermail/awt-dev/2015-March/008974.html > > OS X build broken by reference to XToolkit > > The bug: https://bugs.openjdk.java.net/browse/JDK-8074921 > The fix: http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/c8156e81be2f > The review: > http://mail.openjdk.java.net/pipermail/awt-dev/2015-March/009022.html > From rob.mckenna at oracle.com Thu Mar 26 17:28:00 2015 From: rob.mckenna at oracle.com (Rob McKenna) Date: Thu, 26 Mar 2015 17:28:00 +0000 Subject: [8u60] Request for approval 8072088: [PIT] NPE in DnD tests apparently because of the fix to JDK-8061636 In-Reply-To: <55143CA6.7000207@oracle.com> References: <55143CA6.7000207@oracle.com> Message-ID: <551441A0.8020907@oracle.com> Approved. -Rob On 26/03/15 17:06, Alexander Zvegintsev wrote: > Hello, > > This is a request for approval to backport a fix from the jdk9 to the > jdk8u-dev repository. > Changes applies cleanly to jdk8u-dev after path reshuffling. > > The bug: https://bugs.openjdk.java.net/browse/JDK-8072088 > The fix: http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/d96e70a49103 > The review: > http://mail.openjdk.java.net/pipermail/awt-dev/2015-February/008903.html > From hannes.wallnoefer at oracle.com Fri Mar 27 14:53:10 2015 From: hannes.wallnoefer at oracle.com (Hannes Wallnoefer) Date: Fri, 27 Mar 2015 15:53:10 +0100 Subject: [8u60] Request for approval for JDK-8075366 and JDK-8075231 Message-ID: <55156ED6.8070100@oracle.com> Please approve backport of JDK-8075366 and JDK-8075231 to 8u60: Bug: https://bugs.openjdk.java.net/browse/JDK-8075366 Webrev: http://cr.openjdk.java.net/~hannesw/8075366/ Review thread: http://mail.openjdk.java.net/pipermail/nashorn-dev/2015-March/004404.html Bug: https://bugs.openjdk.java.net/browse/JDK-8075231 Webrev: http://cr.openjdk.java.net/~hannesw/8075231/ Review thread: http://mail.openjdk.java.net/pipermail/nashorn-dev/2015-March/004405.html The changes apply cleanly to 8u-dev after reshuffling path names. Thanks, Hannes From sean.coffey at oracle.com Fri Mar 27 15:27:38 2015 From: sean.coffey at oracle.com (=?UTF-8?B?U2XDoW4gQ29mZmV5?=) Date: Fri, 27 Mar 2015 15:27:38 +0000 Subject: [8u60] Request for approval for JDK-8075366 and JDK-8075231 In-Reply-To: <55156ED6.8070100@oracle.com> References: <55156ED6.8070100@oracle.com> Message-ID: <551576EA.1040906@oracle.com> Approved. regards, Sean. On 27/03/15 14:53, Hannes Wallnoefer wrote: > Please approve backport of JDK-8075366 and JDK-8075231 to 8u60: > > Bug: https://bugs.openjdk.java.net/browse/JDK-8075366 > Webrev: http://cr.openjdk.java.net/~hannesw/8075366/ > Review thread: > http://mail.openjdk.java.net/pipermail/nashorn-dev/2015-March/004404.html > > Bug: https://bugs.openjdk.java.net/browse/JDK-8075231 > Webrev: http://cr.openjdk.java.net/~hannesw/8075231/ > Review thread: > http://mail.openjdk.java.net/pipermail/nashorn-dev/2015-March/004405.html > > The changes apply cleanly to 8u-dev after reshuffling path names. > > Thanks, > Hannes From alejandro.murillo at oracle.com Fri Mar 27 22:24:28 2015 From: alejandro.murillo at oracle.com (Alejandro E Murillo) Date: Fri, 27 Mar 2015 16:24:28 -0600 Subject: [8u60] Request for approval for bulk integration of hs25.60-b09 Message-ID: <5515D89C.9010800@oracle.com> Requesting approval to integrate hs25.60-b09 into jdk8u60-b09. A webrev is available at: http://cr.openjdk.java.net/~amurillo/8u60/hs25.60-b09-jdk8u60-b09.webrev/ Pre-integration testing is in progress; the integration will proceed only after SQE has analyzed the results and approved. The fixes in the proposed integration are below. All have undergone nightly testing and are already in a jdk9 repository. 7127066: Class verifier accepts an invalid class file 8013942: JSR 292: assert(type() == T_OBJECT) failed: type check 8027962: Per-phase timing measurements for strong roots processing 8051837: Remove temporary G1UseParallelRSetUpdating and G1UseParallelRSetScanning flags 8053998: Hot card cache flush chunk size too coarse grained 8055416: Several vm/gc/heap/summary "After GC" events emitted for the same GC ID 8058801: G1TraceReclaimDeadHumongousObjectsAtYoungGC only prints humongous object liveness output when there is at least one candidate humongous object 8060025: Object copy time regressions after JDK-8031323 and JDK-8057536 8061630: G1 iterates over JNIHandles two times 8064473: Improved handling of age during object copy in G1 8065358: Refactor G1s usage of save_marks and reduce related races 8067655: Clean up G1 remembered set oop iteration 8067923: AIX: link libjvm.so with -bernotok to detect missing symbols at build time and suppress warning 1540-1639 8069273: Decrease Hot Card Cache Lock contention 8069760: When iterating over a card, G1 often iterates over much more references than are contained in the card 8072740: move closed jvm.cfg files out of open repo 8074037: Refactor the G1GCPhaseTime logging to make it easier to add new phases 8075118: JVM stuck in infinite loop during verification 8075210: Refactor strong root processing in order to allow G1 to evolve separately from GenCollectedHeap 8075615: new hotspot build - hs25.60-b09 8075858: AIX: clean-up HotSpot make files -- Alejandro From rob.mckenna at oracle.com Fri Mar 27 22:45:54 2015 From: rob.mckenna at oracle.com (Rob McKenna) Date: Fri, 27 Mar 2015 22:45:54 +0000 Subject: [8u60] Request for approval for bulk integration of hs25.60-b09 In-Reply-To: <5515D89C.9010800@oracle.com> References: <5515D89C.9010800@oracle.com> Message-ID: <5515DDA2.40908@oracle.com> Approved. -Rob On 27/03/15 22:24, Alejandro E Murillo wrote: > Requesting approval to integrate hs25.60-b09 into jdk8u60-b09. > > A webrev is available at: > > http://cr.openjdk.java.net/~amurillo/8u60/hs25.60-b09-jdk8u60-b09.webrev/ > > Pre-integration testing is in progress; the integration will proceed > only after SQE has analyzed the results and approved. > > The fixes in the proposed integration are below. All have undergone > nightly testing and are already in a jdk9 repository. > > 7127066: Class verifier accepts an invalid class file > 8013942: JSR 292: assert(type() == T_OBJECT) failed: type check > 8027962: Per-phase timing measurements for strong roots processing > 8051837: Remove temporary G1UseParallelRSetUpdating and > G1UseParallelRSetScanning flags > 8053998: Hot card cache flush chunk size too coarse grained > 8055416: Several vm/gc/heap/summary "After GC" events emitted for the > same GC ID > 8058801: G1TraceReclaimDeadHumongousObjectsAtYoungGC only prints > humongous object liveness output when there is at least one candidate > humongous object > 8060025: Object copy time regressions after JDK-8031323 and JDK-8057536 > 8061630: G1 iterates over JNIHandles two times > 8064473: Improved handling of age during object copy in G1 > 8065358: Refactor G1s usage of save_marks and reduce related races > 8067655: Clean up G1 remembered set oop iteration > 8067923: AIX: link libjvm.so with -bernotok to detect missing symbols > at build time and suppress warning 1540-1639 > 8069273: Decrease Hot Card Cache Lock contention > 8069760: When iterating over a card, G1 often iterates over much more > references than are contained in the card > 8072740: move closed jvm.cfg files out of open repo > 8074037: Refactor the G1GCPhaseTime logging to make it easier to add > new phases > 8075118: JVM stuck in infinite loop during verification > 8075210: Refactor strong root processing in order to allow G1 to > evolve separately from GenCollectedHeap > 8075615: new hotspot build - hs25.60-b09 > 8075858: AIX: clean-up HotSpot make files > From peter.brunet at oracle.com Sat Mar 28 00:30:30 2015 From: peter.brunet at oracle.com (Pete Brunet) Date: Fri, 27 Mar 2015 19:30:30 -0500 Subject: RfR and RfA: Backport of JDK-8055831 Open Source Java Access Bridge Message-ID: <5515F626.30808@oracle.com> Please review and approve a backport of JDK-8055831 to open source the Java Access Bridge. The original patch has been pushed to 9 and the backport now needs to be pushed to 8u. Due to the significant directory structure changes in 9 due to modularization I could not simply apply the patch. However the effect is functionally the same. Here are the changes: - moved the JAB files from closed to open - change the copyright headers to open style - adjust the make files accordingly - create javadoc for com.sun.java.accessibility.util; creates it in docs/jre/api/accessibility/jaccess/spec - removed extraneous unused files; these are tools inadvertently picked up in 7u6 when the JAB was first integrated and which will be added later - removed some dead code - cleaned up comments/documentation - cleaned up some code indentation issues - in package-private EventQueueMonitor.addTopLevelWindow the return value was not used so I changed the return from boolean to void The open part of the patch is here: http://cr.openjdk.java.net/~ptbrunet/JDK-8076182/webrev.00/ (I see I have five copyright dates to fix in jdk/make.) There is a closed part but it's just the removal of the source files from the closed repository. The patch has been through a successful 32/64 bit JPRT build and subsequent testing on my machine This patch needs to be pushed by Tuesday so I'll appreciate your review. Thanks, Pete From erik.joelsson at oracle.com Mon Mar 30 07:43:55 2015 From: erik.joelsson at oracle.com (Erik Joelsson) Date: Mon, 30 Mar 2015 09:43:55 +0200 Subject: RfR and RfA: Backport of JDK-8055831 Open Source Java Access Bridge In-Reply-To: <5515F626.30808@oracle.com> References: <5515F626.30808@oracle.com> Message-ID: <5518FEBB.7020103@oracle.com> Hello, In jdk/make/CompileJavaClasses.gmk: You removed $(JDK_TOPDIR)/src/closed/$(OPENJDK_TARGET_OS_API_DIR)/classes, but for linux and solaris, that source dir is still needed. Is it giving you an error when the directory doesn't exist? In that case you will need to make it conditional on platform. jdk/make/CompileLaunchers.gmk: If jabswitch is open source, then it shouldn't be built under the condition "ifndef OPENJDK". jdk/make/CopyFiles.gmk: ifndef OPENJDK jdk/make/gensrc/GensrcMisc.gmk: ifndef OPENJDK jdk/make/lib/PlatformLibraries.gmk: ifndef OPENJDK I would recommend getting an open only forest, apply your patch and verify that you get everything built properly. /Erik On 2015-03-28 01:30, Pete Brunet wrote: > Please review and approve a backport of JDK-8055831 to open source the > Java Access Bridge. The original patch has been pushed to 9 and the > backport now needs to be pushed to 8u. > > Due to the significant directory structure changes in 9 due to > modularization I could not simply apply the patch. However the effect > is functionally the same. Here are the changes: > > - moved the JAB files from closed to open > - change the copyright headers to open style > - adjust the make files accordingly > - create javadoc for com.sun.java.accessibility.util; creates it in > docs/jre/api/accessibility/jaccess/spec > - removed extraneous unused files; these are tools inadvertently picked > up in 7u6 when the JAB was first integrated and which will be added later > - removed some dead code > - cleaned up comments/documentation > - cleaned up some code indentation issues > - in package-private EventQueueMonitor.addTopLevelWindow the return > value was not used so I changed the return from boolean to void > > The open part of the patch is here: > http://cr.openjdk.java.net/~ptbrunet/JDK-8076182/webrev.00/ > > (I see I have five copyright dates to fix in jdk/make.) > > There is a closed part but it's just the removal of the source files > from the closed repository. > > The patch has been through a successful 32/64 bit JPRT build and > subsequent testing on my machine > > This patch needs to be pushed by Tuesday so I'll appreciate your review. > > Thanks, Pete From masayoshi.okutsu at oracle.com Mon Mar 30 08:38:31 2015 From: masayoshi.okutsu at oracle.com (Masayoshi Okutsu) Date: Mon, 30 Mar 2015 17:38:31 +0900 Subject: [8u60] Request for approval: 8075548: SimpleDateFormat formatting of "LLLL" in English is incorrect; should be identical to "MMMM" Message-ID: <55190B87.6010008@oracle.com> Hello, Could you please approve backport of the fix for 8075548 to jdk8u-dev? The fix is identical to the JDK 9 one. Bug: https://bugs.openjdk.java.net/browse/JDK-8075548 Webrev: http://cr.openjdk.java.net/~okutsu/8u/8075548/webrev.00/ Review thread: http://mail.openjdk.java.net/pipermail/i18n-dev/2015-March/001656.html JDK9 changeset: http://hg.openjdk.java.net/jdk9/dev/jdk/rev/0c0c2f87eeb2 Thanks, Masayoshi From sean.coffey at oracle.com Mon Mar 30 08:43:04 2015 From: sean.coffey at oracle.com (=?UTF-8?B?U2XDoW4gQ29mZmV5?=) Date: Mon, 30 Mar 2015 09:43:04 +0100 Subject: [8u60] Request for approval: 8075548: SimpleDateFormat formatting of "LLLL" in English is incorrect; should be identical to "MMMM" In-Reply-To: <55190B87.6010008@oracle.com> References: <55190B87.6010008@oracle.com> Message-ID: <55190C98.7030208@oracle.com> Approved. regards, Sean. On 30/03/2015 09:38, Masayoshi Okutsu wrote: > Hello, > > Could you please approve backport of the fix for 8075548 to jdk8u-dev? > The fix is identical to the JDK 9 one. > > Bug: > https://bugs.openjdk.java.net/browse/JDK-8075548 > Webrev: > http://cr.openjdk.java.net/~okutsu/8u/8075548/webrev.00/ > Review thread: > http://mail.openjdk.java.net/pipermail/i18n-dev/2015-March/001656.html > JDK9 changeset: > http://hg.openjdk.java.net/jdk9/dev/jdk/rev/0c0c2f87eeb2 > > Thanks, > Masayoshi > From alexandr.scherbatiy at oracle.com Mon Mar 30 12:17:43 2015 From: alexandr.scherbatiy at oracle.com (Alexander Scherbatiy) Date: Mon, 30 Mar 2015 15:17:43 +0300 Subject: [8u60] Request for approval: 8039926 -spash: can't be combined with -xStartOnFirstThread since JDK 7 Message-ID: <55193EE7.70304@oracle.com> Hello, Could you approve the direct backport of the fix to JDK 8u-dev. The bug: https://bugs.openjdk.java.net/browse/JDK-8039926 The webrev: http://cr.openjdk.java.net/~pchelko/9/8039926/webrev.00/ The review thread: http://mail.openjdk.java.net/pipermail/awt-dev/2014-April/007559.html The JDK 9 changeset:http://hg.openjdk.java.net/jdk9/client/jdk/rev/a673ffda4f64 Thanks, Alexandr. From alexandr.scherbatiy at oracle.com Mon Mar 30 12:21:14 2015 From: alexandr.scherbatiy at oracle.com (Alexander Scherbatiy) Date: Mon, 30 Mar 2015 15:21:14 +0300 Subject: [8u60] Request for approval: 8075244 [macosx] The fix for JDK-8043869 should be reworked Message-ID: <55193FBA.4010403@oracle.com> Hello, Could you approve the direct backport of the fix to JDK 8u-dev. The bug: https://bugs.openjdk.java.net/browse/JDK-8075244 The webrev: http://cr.openjdk.java.net/~alexsch/8075244/webrev.03 The review thread: http://mail.openjdk.java.net/pipermail/awt-dev/2015-March/009101.html The JDK 9 changeset: http://hg.openjdk.java.net/jdk9/client/jdk/rev/2ff00fc904dd Thanks, Alexandr. From sean.coffey at oracle.com Mon Mar 30 12:38:05 2015 From: sean.coffey at oracle.com (=?windows-1252?Q?Se=E1n_Coffey?=) Date: Mon, 30 Mar 2015 13:38:05 +0100 Subject: [8u60] Request for approval: 8039926 -spash: can't be combined with -xStartOnFirstThread since JDK 7 In-Reply-To: <55193EE7.70304@oracle.com> References: <55193EE7.70304@oracle.com> Message-ID: <551943AD.7070200@oracle.com> Approved. Please add a suitable noreg- label to the bug report. regards, Sean. On 30/03/15 13:17, Alexander Scherbatiy wrote: > > Hello, > > Could you approve the direct backport of the fix to JDK 8u-dev. > > The bug: https://bugs.openjdk.java.net/browse/JDK-8039926 > The webrev: http://cr.openjdk.java.net/~pchelko/9/8039926/webrev.00/ > The review thread: > http://mail.openjdk.java.net/pipermail/awt-dev/2014-April/007559.html > The JDK 9 > changeset:http://hg.openjdk.java.net/jdk9/client/jdk/rev/a673ffda4f64 > > Thanks, > Alexandr. > From sean.coffey at oracle.com Mon Mar 30 12:39:06 2015 From: sean.coffey at oracle.com (=?windows-1252?Q?Se=E1n_Coffey?=) Date: Mon, 30 Mar 2015 13:39:06 +0100 Subject: [8u60] Request for approval: 8075244 [macosx] The fix for JDK-8043869 should be reworked In-Reply-To: <55193FBA.4010403@oracle.com> References: <55193FBA.4010403@oracle.com> Message-ID: <551943EA.6090100@oracle.com> Approved. regards, Sean. On 30/03/15 13:21, Alexander Scherbatiy wrote: > > Hello, > > Could you approve the direct backport of the fix to JDK 8u-dev. > > The bug: https://bugs.openjdk.java.net/browse/JDK-8075244 > The webrev: http://cr.openjdk.java.net/~alexsch/8075244/webrev.03 > The review thread: > http://mail.openjdk.java.net/pipermail/awt-dev/2015-March/009101.html > The JDK 9 changeset: > http://hg.openjdk.java.net/jdk9/client/jdk/rev/2ff00fc904dd > > Thanks, > Alexandr. > From alexandr.scherbatiy at oracle.com Mon Mar 30 13:29:17 2015 From: alexandr.scherbatiy at oracle.com (Alexander Scherbatiy) Date: Mon, 30 Mar 2015 16:29:17 +0300 Subject: [8u60] Request for approval: 8033000 No Horizontal Mouse Wheel Support In BasicScrollPaneUI Message-ID: <55194FAD.2040804@oracle.com> Hello, Could you approve the direct backport of the fix to JDK 8u-dev. The bug: https://bugs.openjdk.java.net/browse/JDK-8033000 The webrev: http://cr.openjdk.java.net/~alexsch/8033000/webrev.00 The review thread: http://mail.openjdk.java.net/pipermail/swing-dev/2015-March/004312.html The JDK 9 changeset: http://hg.openjdk.java.net/jdk9/client/jdk/rev/c2acbd0292f3 Thanks, Alexandr. From peter.brunet at oracle.com Mon Mar 30 13:39:28 2015 From: peter.brunet at oracle.com (Pete Brunet) Date: Mon, 30 Mar 2015 08:39:28 -0500 Subject: RfR and RfA: Backport of JDK-8055831 Open Source Java Access Bridge In-Reply-To: <5518FEBB.7020103@oracle.com> References: <5515F626.30808@oracle.com> <5518FEBB.7020103@oracle.com> Message-ID: <55195210.6090905@oracle.com> Thanks Erik, I will work on this. I appreciate your review. -Pete On 3/30/15 2:43 AM, Erik Joelsson wrote: > Hello, > > In jdk/make/CompileJavaClasses.gmk: > > You removed > $(JDK_TOPDIR)/src/closed/$(OPENJDK_TARGET_OS_API_DIR)/classes, but for > linux and solaris, that source dir is still needed. Is it giving you > an error when the directory doesn't exist? In that case you will need > to make it conditional on platform. > > jdk/make/CompileLaunchers.gmk: > > If jabswitch is open source, then it shouldn't be built under the > condition "ifndef OPENJDK". > > jdk/make/CopyFiles.gmk: > > ifndef OPENJDK > > jdk/make/gensrc/GensrcMisc.gmk: > > ifndef OPENJDK > > jdk/make/lib/PlatformLibraries.gmk: > > ifndef OPENJDK > > > I would recommend getting an open only forest, apply your patch and > verify that you get everything built properly. > > /Erik > > On 2015-03-28 01:30, Pete Brunet wrote: >> Please review and approve a backport of JDK-8055831 to open source the >> Java Access Bridge. The original patch has been pushed to 9 and the >> backport now needs to be pushed to 8u. >> >> Due to the significant directory structure changes in 9 due to >> modularization I could not simply apply the patch. However the effect >> is functionally the same. Here are the changes: >> >> - moved the JAB files from closed to open >> - change the copyright headers to open style >> - adjust the make files accordingly >> - create javadoc for com.sun.java.accessibility.util; creates it in >> docs/jre/api/accessibility/jaccess/spec >> - removed extraneous unused files; these are tools inadvertently picked >> up in 7u6 when the JAB was first integrated and which will be added later >> - removed some dead code >> - cleaned up comments/documentation >> - cleaned up some code indentation issues >> - in package-private EventQueueMonitor.addTopLevelWindow the return >> value was not used so I changed the return from boolean to void >> >> The open part of the patch is here: >> http://cr.openjdk.java.net/~ptbrunet/JDK-8076182/webrev.00/ >> >> (I see I have five copyright dates to fix in jdk/make.) >> >> There is a closed part but it's just the removal of the source files >> from the closed repository. >> >> The patch has been through a successful 32/64 bit JPRT build and >> subsequent testing on my machine >> >> This patch needs to be pushed by Tuesday so I'll appreciate your review. >> >> Thanks, Pete > From rob.mckenna at oracle.com Mon Mar 30 14:11:48 2015 From: rob.mckenna at oracle.com (Rob McKenna) Date: Mon, 30 Mar 2015 15:11:48 +0100 Subject: [8u60] Request for approval: 8033000 No Horizontal Mouse Wheel Support In BasicScrollPaneUI In-Reply-To: <55194FAD.2040804@oracle.com> References: <55194FAD.2040804@oracle.com> Message-ID: <551959A4.6040709@oracle.com> Approved. -Rob On 30/03/15 14:29, Alexander Scherbatiy wrote: > > Hello, > > Could you approve the direct backport of the fix to JDK 8u-dev. > > The bug: https://bugs.openjdk.java.net/browse/JDK-8033000 > The webrev: http://cr.openjdk.java.net/~alexsch/8033000/webrev.00 > The review thread: > http://mail.openjdk.java.net/pipermail/swing-dev/2015-March/004312.html > The JDK 9 changeset: > http://hg.openjdk.java.net/jdk9/client/jdk/rev/c2acbd0292f3 > > Thanks, > Alexandr. > From anton.nashatyrev at oracle.com Mon Mar 30 14:56:13 2015 From: anton.nashatyrev at oracle.com (Anton Nashatyrev) Date: Mon, 30 Mar 2015 17:56:13 +0300 Subject: [8u-dev] Backport request: 8074481: [macosx] Menu items are appearing on top of other windows Message-ID: <5519640D.9050602@oracle.com> Hello, please approve the fix backport from jdk9 to jdk8u-dev. The patch was applied without any modifications. Bug: https://bugs.openjdk.java.net/browse/JDK-8074481 Jdk9 Changeset: http://hg.openjdk.java.net/jdk9/client/jdk/rev/bb4f4297dda7 Jdk9 Review Thread: http://mail.openjdk.java.net/pipermail/awt-dev/2015-March/009079.html Thanks! Anton. From rob.mckenna at oracle.com Mon Mar 30 15:00:37 2015 From: rob.mckenna at oracle.com (Rob McKenna) Date: Mon, 30 Mar 2015 16:00:37 +0100 Subject: [8u-dev] Backport request: 8074481: [macosx] Menu items are appearing on top of other windows In-Reply-To: <5519640D.9050602@oracle.com> References: <5519640D.9050602@oracle.com> Message-ID: <55196515.4010305@oracle.com> Approved. -Rob On 30/03/15 15:56, Anton Nashatyrev wrote: > Hello, > > please approve the fix backport from jdk9 to jdk8u-dev. The patch > was applied without any modifications. > > Bug: https://bugs.openjdk.java.net/browse/JDK-8074481 > Jdk9 Changeset: > http://hg.openjdk.java.net/jdk9/client/jdk/rev/bb4f4297dda7 > Jdk9 Review Thread: > http://mail.openjdk.java.net/pipermail/awt-dev/2015-March/009079.html > > Thanks! > Anton. From anton.nashatyrev at oracle.com Mon Mar 30 15:59:54 2015 From: anton.nashatyrev at oracle.com (Anton Nashatyrev) Date: Mon, 30 Mar 2015 18:59:54 +0300 Subject: [8u-dev] Backport request: 8071668: [macosx] Clipboard does not work with 3rd parties Clipboard Managers Message-ID: <551972FA.5020406@oracle.com> Hello, please approve the fix backport from jdk9 to jdk8u-dev. One file (CClipboard.m) couldn't be patched strictly since been refactored in the Jdk9, but the changes are essentially the same. Bug: https://bugs.openjdk.java.net/browse/JDK-8071668 Jdk8 Webrev: http://cr.openjdk.java.net/~anashaty/8071668/8/webrev.00/ Jdk9 Changeset: http://hg.openjdk.java.net/jdk9/client/jdk/rev/51d72921a268 Jdk9 Review Thread: http://mail.openjdk.java.net/pipermail/awt-dev/2015-March/009136.html Thanks! Anton. From peter.brunet at oracle.com Mon Mar 30 16:10:13 2015 From: peter.brunet at oracle.com (Pete Brunet) Date: Mon, 30 Mar 2015 11:10:13 -0500 Subject: RfR and RfA: Backport of JDK-8055831 Open Source Java Access Bridge In-Reply-To: <5518FEBB.7020103@oracle.com> References: <5515F626.30808@oracle.com> <5518FEBB.7020103@oracle.com> Message-ID: <55197565.2090706@oracle.com> Erik, I made the changes that you pointed out. Builds worked OK for local closed and open. I also started a JPRT build for all platforms. That's 7 targets and since that will take a while I wanted to publish the new patch before you left for the day: http://cr.openjdk.java.net/~ptbrunet/JDK-8076182/webrev.01/ Pete On 3/30/15 2:43 AM, Erik Joelsson wrote: > Hello, > > In jdk/make/CompileJavaClasses.gmk: > > You removed > $(JDK_TOPDIR)/src/closed/$(OPENJDK_TARGET_OS_API_DIR)/classes, but for > linux and solaris, that source dir is still needed. Is it giving you > an error when the directory doesn't exist? In that case you will need > to make it conditional on platform. > > jdk/make/CompileLaunchers.gmk: > > If jabswitch is open source, then it shouldn't be built under the > condition "ifndef OPENJDK". > > jdk/make/CopyFiles.gmk: > > ifndef OPENJDK > > jdk/make/gensrc/GensrcMisc.gmk: > > ifndef OPENJDK > > jdk/make/lib/PlatformLibraries.gmk: > > ifndef OPENJDK > > > I would recommend getting an open only forest, apply your patch and > verify that you get everything built properly. > > /Erik > > On 2015-03-28 01:30, Pete Brunet wrote: >> Please review and approve a backport of JDK-8055831 to open source the >> Java Access Bridge. The original patch has been pushed to 9 and the >> backport now needs to be pushed to 8u. >> >> Due to the significant directory structure changes in 9 due to >> modularization I could not simply apply the patch. However the effect >> is functionally the same. Here are the changes: >> >> - moved the JAB files from closed to open >> - change the copyright headers to open style >> - adjust the make files accordingly >> - create javadoc for com.sun.java.accessibility.util; creates it in >> docs/jre/api/accessibility/jaccess/spec >> - removed extraneous unused files; these are tools inadvertently picked >> up in 7u6 when the JAB was first integrated and which will be added later >> - removed some dead code >> - cleaned up comments/documentation >> - cleaned up some code indentation issues >> - in package-private EventQueueMonitor.addTopLevelWindow the return >> value was not used so I changed the return from boolean to void >> >> The open part of the patch is here: >> http://cr.openjdk.java.net/~ptbrunet/JDK-8076182/webrev.00/ >> >> (I see I have five copyright dates to fix in jdk/make.) >> >> There is a closed part but it's just the removal of the source files >> from the closed repository. >> >> The patch has been through a successful 32/64 bit JPRT build and >> subsequent testing on my machine >> >> This patch needs to be pushed by Tuesday so I'll appreciate your review. >> >> Thanks, Pete > From sean.coffey at oracle.com Mon Mar 30 16:18:39 2015 From: sean.coffey at oracle.com (=?UTF-8?B?U2XDoW4gQ29mZmV5?=) Date: Mon, 30 Mar 2015 17:18:39 +0100 Subject: [8u-dev] Backport request: 8071668: [macosx] Clipboard does not work with 3rd parties Clipboard Managers In-Reply-To: <551972FA.5020406@oracle.com> References: <551972FA.5020406@oracle.com> Message-ID: <5519775F.1050801@oracle.com> Approved. regards, Sean. On 30/03/15 16:59, Anton Nashatyrev wrote: > Hello, > > please approve the fix backport from jdk9 to jdk8u-dev. > One file (CClipboard.m) couldn't be patched strictly since been > refactored in the Jdk9, but the changes are essentially the same. > > Bug: https://bugs.openjdk.java.net/browse/JDK-8071668 > Jdk8 Webrev: http://cr.openjdk.java.net/~anashaty/8071668/8/webrev.00/ > > Jdk9 Changeset: > http://hg.openjdk.java.net/jdk9/client/jdk/rev/51d72921a268 > Jdk9 Review Thread: > http://mail.openjdk.java.net/pipermail/awt-dev/2015-March/009136.html > > Thanks! > Anton. From jan.lahoda at oracle.com Mon Mar 30 17:03:56 2015 From: jan.lahoda at oracle.com (Jan Lahoda) Date: Mon, 30 Mar 2015 19:03:56 +0200 Subject: [8u-dev] request for review and approval to backport JDK-8037546, JDK-8054220 and JDK-8058227 In-Reply-To: <54DD181C.30206@oracle.com> References: <54DD181C.30206@oracle.com> Message-ID: <551981FC.9000205@oracle.com> Hello, I'd like to ask for an approval to backport the following two fixes to 8u-dev: Bug: https://bugs.openjdk.java.net/browse/JDK-8037546 jdk9 fix: http://hg.openjdk.java.net/jdk9/dev/langtools/rev/441711fd360d (this fix is a pre-requisite for the following fix) Bugs: https://bugs.openjdk.java.net/browse/JDK-8054220 https://bugs.openjdk.java.net/browse/JDK-8058227 jdk9 fix: http://hg.openjdk.java.net/jdk9/dev/langtools/rev/c4a6ba06abeb Both fixes apply cleanly to 8u-dev/langtools after unshuffling. Thanks, Jan From peter.brunet at oracle.com Mon Mar 30 18:39:27 2015 From: peter.brunet at oracle.com (Pete Brunet) Date: Mon, 30 Mar 2015 13:39:27 -0500 Subject: RfR and RfA: Backport of JDK-8055831 Open Source Java Access Bridge In-Reply-To: <55197565.2090706@oracle.com> References: <5515F626.30808@oracle.com> <5518FEBB.7020103@oracle.com> <55197565.2090706@oracle.com> Message-ID: <5519985F.9030907@oracle.com> p.s. The 7 JPRT builds ran OK: 2 win, 2 linux, 2 solaris, 1 mac On 3/30/15 11:10 AM, Pete Brunet wrote: > Erik, I made the changes that you pointed out. Builds worked OK for > local closed and open. I also started a JPRT build for all > platforms. That's 7 targets and since that will take a while I wanted > to publish the new patch before you left for the day: > > http://cr.openjdk.java.net/~ptbrunet/JDK-8076182/webrev.01/ > > Pete > > On 3/30/15 2:43 AM, Erik Joelsson wrote: >> Hello, >> >> In jdk/make/CompileJavaClasses.gmk: >> >> You removed >> $(JDK_TOPDIR)/src/closed/$(OPENJDK_TARGET_OS_API_DIR)/classes, but >> for linux and solaris, that source dir is still needed. Is it giving >> you an error when the directory doesn't exist? In that case you will >> need to make it conditional on platform. >> >> jdk/make/CompileLaunchers.gmk: >> >> If jabswitch is open source, then it shouldn't be built under the >> condition "ifndef OPENJDK". >> >> jdk/make/CopyFiles.gmk: >> >> ifndef OPENJDK >> >> jdk/make/gensrc/GensrcMisc.gmk: >> >> ifndef OPENJDK >> >> jdk/make/lib/PlatformLibraries.gmk: >> >> ifndef OPENJDK >> >> >> I would recommend getting an open only forest, apply your patch and >> verify that you get everything built properly. >> >> /Erik >> >> On 2015-03-28 01:30, Pete Brunet wrote: >>> Please review and approve a backport of JDK-8055831 to open source the >>> Java Access Bridge. The original patch has been pushed to 9 and the >>> backport now needs to be pushed to 8u. >>> >>> Due to the significant directory structure changes in 9 due to >>> modularization I could not simply apply the patch. However the effect >>> is functionally the same. Here are the changes: >>> >>> - moved the JAB files from closed to open >>> - change the copyright headers to open style >>> - adjust the make files accordingly >>> - create javadoc for com.sun.java.accessibility.util; creates it in >>> docs/jre/api/accessibility/jaccess/spec >>> - removed extraneous unused files; these are tools inadvertently picked >>> up in 7u6 when the JAB was first integrated and which will be added later >>> - removed some dead code >>> - cleaned up comments/documentation >>> - cleaned up some code indentation issues >>> - in package-private EventQueueMonitor.addTopLevelWindow the return >>> value was not used so I changed the return from boolean to void >>> >>> The open part of the patch is here: >>> http://cr.openjdk.java.net/~ptbrunet/JDK-8076182/webrev.00/ >>> >>> (I see I have five copyright dates to fix in jdk/make.) >>> >>> There is a closed part but it's just the removal of the source files >>> from the closed repository. >>> >>> The patch has been through a successful 32/64 bit JPRT build and >>> subsequent testing on my machine >>> >>> This patch needs to be pushed by Tuesday so I'll appreciate your review. >>> >>> Thanks, Pete >> > From sean.coffey at oracle.com Mon Mar 30 19:05:20 2015 From: sean.coffey at oracle.com (=?UTF-8?B?U2XDoW4gQ29mZmV5?=) Date: Mon, 30 Mar 2015 20:05:20 +0100 Subject: RfR and RfA: Backport of JDK-8055831 Open Source Java Access Bridge In-Reply-To: <5519985F.9030907@oracle.com> References: <5515F626.30808@oracle.com> <5518FEBB.7020103@oracle.com> <55197565.2090706@oracle.com> <5519985F.9030907@oracle.com> Message-ID: <55199E70.1000202@oracle.com> Pete, I'm not a reviewer for this but I notice that some files have the incorrect copyright header (PROPRIETARY/CONFIDENTIAL) - Please correct that before any push. I think it's also safer if you run both an OpenJDK build and a closed/open src build through JPRT. It'll handle both. regards, Sean. On 30/03/2015 19:39, Pete Brunet wrote: > p.s. The 7 JPRT builds ran OK: 2 win, 2 linux, 2 solaris, 1 mac > > On 3/30/15 11:10 AM, Pete Brunet wrote: >> Erik, I made the changes that you pointed out. Builds worked OK for >> local closed and open. I also started a JPRT build for all >> platforms. That's 7 targets and since that will take a while I wanted >> to publish the new patch before you left for the day: >> >> http://cr.openjdk.java.net/~ptbrunet/JDK-8076182/webrev.01/ >> >> Pete >> >> On 3/30/15 2:43 AM, Erik Joelsson wrote: >>> Hello, >>> >>> In jdk/make/CompileJavaClasses.gmk: >>> >>> You removed >>> $(JDK_TOPDIR)/src/closed/$(OPENJDK_TARGET_OS_API_DIR)/classes, but >>> for linux and solaris, that source dir is still needed. Is it giving >>> you an error when the directory doesn't exist? In that case you will >>> need to make it conditional on platform. >>> >>> jdk/make/CompileLaunchers.gmk: >>> >>> If jabswitch is open source, then it shouldn't be built under the >>> condition "ifndef OPENJDK". >>> >>> jdk/make/CopyFiles.gmk: >>> >>> ifndef OPENJDK >>> >>> jdk/make/gensrc/GensrcMisc.gmk: >>> >>> ifndef OPENJDK >>> >>> jdk/make/lib/PlatformLibraries.gmk: >>> >>> ifndef OPENJDK >>> >>> >>> I would recommend getting an open only forest, apply your patch and >>> verify that you get everything built properly. >>> >>> /Erik >>> >>> On 2015-03-28 01:30, Pete Brunet wrote: >>>> Please review and approve a backport of JDK-8055831 to open source the >>>> Java Access Bridge. The original patch has been pushed to 9 and the >>>> backport now needs to be pushed to 8u. >>>> >>>> Due to the significant directory structure changes in 9 due to >>>> modularization I could not simply apply the patch. However the effect >>>> is functionally the same. Here are the changes: >>>> >>>> - moved the JAB files from closed to open >>>> - change the copyright headers to open style >>>> - adjust the make files accordingly >>>> - create javadoc for com.sun.java.accessibility.util; creates it in >>>> docs/jre/api/accessibility/jaccess/spec >>>> - removed extraneous unused files; these are tools inadvertently picked >>>> up in 7u6 when the JAB was first integrated and which will be added later >>>> - removed some dead code >>>> - cleaned up comments/documentation >>>> - cleaned up some code indentation issues >>>> - in package-private EventQueueMonitor.addTopLevelWindow the return >>>> value was not used so I changed the return from boolean to void >>>> >>>> The open part of the patch is here: >>>> http://cr.openjdk.java.net/~ptbrunet/JDK-8076182/webrev.00/ >>>> >>>> (I see I have five copyright dates to fix in jdk/make.) >>>> >>>> There is a closed part but it's just the removal of the source files >>>> from the closed repository. >>>> >>>> The patch has been through a successful 32/64 bit JPRT build and >>>> subsequent testing on my machine >>>> >>>> This patch needs to be pushed by Tuesday so I'll appreciate your review. >>>> >>>> Thanks, Pete From alexander.zvegintsev at oracle.com Mon Mar 30 23:52:51 2015 From: alexander.zvegintsev at oracle.com (Alexander Zvegintsev) Date: Tue, 31 Mar 2015 02:52:51 +0300 Subject: RfR and RfA: Backport of JDK-8055831 Open Source Java Access Bridge In-Reply-To: <55197565.2090706@oracle.com> References: <5515F626.30808@oracle.com> <5518FEBB.7020103@oracle.com> <55197565.2090706@oracle.com> Message-ID: <5519E1D3.1050203@oracle.com> Hello Pete, the AWT part looks good to me (except for confidential copyright headers of course). -- Thanks, Alexander. On 30.03.2015 19:10, Pete Brunet wrote: > Erik, I made the changes that you pointed out. Builds worked OK for > local closed and open. I also started a JPRT build for all platforms. > That's 7 targets and since that will take a while I wanted to publish > the new patch before you left for the day: > > http://cr.openjdk.java.net/~ptbrunet/JDK-8076182/webrev.01/ > > Pete > > On 3/30/15 2:43 AM, Erik Joelsson wrote: >> Hello, >> >> In jdk/make/CompileJavaClasses.gmk: >> >> You removed >> $(JDK_TOPDIR)/src/closed/$(OPENJDK_TARGET_OS_API_DIR)/classes, but for >> linux and solaris, that source dir is still needed. Is it giving you >> an error when the directory doesn't exist? In that case you will need >> to make it conditional on platform. >> >> jdk/make/CompileLaunchers.gmk: >> >> If jabswitch is open source, then it shouldn't be built under the >> condition "ifndef OPENJDK". >> >> jdk/make/CopyFiles.gmk: >> >> ifndef OPENJDK >> >> jdk/make/gensrc/GensrcMisc.gmk: >> >> ifndef OPENJDK >> >> jdk/make/lib/PlatformLibraries.gmk: >> >> ifndef OPENJDK >> >> >> I would recommend getting an open only forest, apply your patch and >> verify that you get everything built properly. >> >> /Erik >> >> On 2015-03-28 01:30, Pete Brunet wrote: >>> Please review and approve a backport of JDK-8055831 to open source the >>> Java Access Bridge. The original patch has been pushed to 9 and the >>> backport now needs to be pushed to 8u. >>> >>> Due to the significant directory structure changes in 9 due to >>> modularization I could not simply apply the patch. However the effect >>> is functionally the same. Here are the changes: >>> >>> - moved the JAB files from closed to open >>> - change the copyright headers to open style >>> - adjust the make files accordingly >>> - create javadoc for com.sun.java.accessibility.util; creates it in >>> docs/jre/api/accessibility/jaccess/spec >>> - removed extraneous unused files; these are tools inadvertently picked >>> up in 7u6 when the JAB was first integrated and which will be added later >>> - removed some dead code >>> - cleaned up comments/documentation >>> - cleaned up some code indentation issues >>> - in package-private EventQueueMonitor.addTopLevelWindow the return >>> value was not used so I changed the return from boolean to void >>> >>> The open part of the patch is here: >>> http://cr.openjdk.java.net/~ptbrunet/JDK-8076182/webrev.00/ >>> >>> (I see I have five copyright dates to fix in jdk/make.) >>> >>> There is a closed part but it's just the removal of the source files >>> from the closed repository. >>> >>> The patch has been through a successful 32/64 bit JPRT build and >>> subsequent testing on my machine >>> >>> This patch needs to be pushed by Tuesday so I'll appreciate your review. >>> >>> Thanks, Pete From peter.brunet at oracle.com Tue Mar 31 00:10:12 2015 From: peter.brunet at oracle.com (Pete Brunet) Date: Mon, 30 Mar 2015 19:10:12 -0500 Subject: RfR and RfA: Backport of JDK-8055831 Open Source Java Access Bridge In-Reply-To: <5519E1D3.1050203@oracle.com> References: <5515F626.30808@oracle.com> <5518FEBB.7020103@oracle.com> <55197565.2090706@oracle.com> <5519E1D3.1050203@oracle.com> Message-ID: <5519E5E4.2090507@oracle.com> Thank you Alexander. On 3/30/15 6:52 PM, Alexander Zvegintsev wrote: > Hello Pete, > > the AWT part looks good to me (except for confidential copyright > headers of course). > > -- > Thanks, > Alexander. > > On 30.03.2015 19:10, Pete Brunet wrote: >> Erik, I made the changes that you pointed out. Builds worked OK for >> local closed and open. I also started a JPRT build for all platforms. >> That's 7 targets and since that will take a while I wanted to publish >> the new patch before you left for the day: >> >> http://cr.openjdk.java.net/~ptbrunet/JDK-8076182/webrev.01/ >> >> Pete >> >> On 3/30/15 2:43 AM, Erik Joelsson wrote: >>> Hello, >>> >>> In jdk/make/CompileJavaClasses.gmk: >>> >>> You removed >>> $(JDK_TOPDIR)/src/closed/$(OPENJDK_TARGET_OS_API_DIR)/classes, but for >>> linux and solaris, that source dir is still needed. Is it giving you >>> an error when the directory doesn't exist? In that case you will need >>> to make it conditional on platform. >>> >>> jdk/make/CompileLaunchers.gmk: >>> >>> If jabswitch is open source, then it shouldn't be built under the >>> condition "ifndef OPENJDK". >>> >>> jdk/make/CopyFiles.gmk: >>> >>> ifndef OPENJDK >>> >>> jdk/make/gensrc/GensrcMisc.gmk: >>> >>> ifndef OPENJDK >>> >>> jdk/make/lib/PlatformLibraries.gmk: >>> >>> ifndef OPENJDK >>> >>> >>> I would recommend getting an open only forest, apply your patch and >>> verify that you get everything built properly. >>> >>> /Erik >>> >>> On 2015-03-28 01:30, Pete Brunet wrote: >>>> Please review and approve a backport of JDK-8055831 to open source the >>>> Java Access Bridge. The original patch has been pushed to 9 and the >>>> backport now needs to be pushed to 8u. >>>> >>>> Due to the significant directory structure changes in 9 due to >>>> modularization I could not simply apply the patch. However the effect >>>> is functionally the same. Here are the changes: >>>> >>>> - moved the JAB files from closed to open >>>> - change the copyright headers to open style >>>> - adjust the make files accordingly >>>> - create javadoc for com.sun.java.accessibility.util; creates it in >>>> docs/jre/api/accessibility/jaccess/spec >>>> - removed extraneous unused files; these are tools inadvertently >>>> picked >>>> up in 7u6 when the JAB was first integrated and which will be added >>>> later >>>> - removed some dead code >>>> - cleaned up comments/documentation >>>> - cleaned up some code indentation issues >>>> - in package-private EventQueueMonitor.addTopLevelWindow the return >>>> value was not used so I changed the return from boolean to void >>>> >>>> The open part of the patch is here: >>>> http://cr.openjdk.java.net/~ptbrunet/JDK-8076182/webrev.00/ >>>> >>>> (I see I have five copyright dates to fix in jdk/make.) >>>> >>>> There is a closed part but it's just the removal of the source files >>>> from the closed repository. >>>> >>>> The patch has been through a successful 32/64 bit JPRT build and >>>> subsequent testing on my machine >>>> >>>> This patch needs to be pushed by Tuesday so I'll appreciate your >>>> review. >>>> >>>> Thanks, Pete > From peter.brunet at oracle.com Tue Mar 31 03:21:42 2015 From: peter.brunet at oracle.com (Pete Brunet) Date: Mon, 30 Mar 2015 22:21:42 -0500 Subject: RfR and RfA: Backport of JDK-8055831 Open Source Java Access Bridge In-Reply-To: <55199E70.1000202@oracle.com> References: <5515F626.30808@oracle.com> <5518FEBB.7020103@oracle.com> <55197565.2090706@oracle.com> <5519985F.9030907@oracle.com> <55199E70.1000202@oracle.com> Message-ID: <551A12C6.70603@oracle.com> Thanks Sean, The JPRT open build pointed out some problems which I have resolved but need Erik's review of those changes which are here: http://cr.openjdk.java.net/~ptbrunet/JDK-8076182/webrev.02/ Alexander Zvegintsev reviewed the source so once Erik finishes his review hopefully you'll be able to approve it for push into 8u-dev. Erik, There are some places where I changed ifndef OpenJDK to ifdef OpenJDK. I didn't think of it until too late in the evening, but I probably should have just removed that. If so, I can do that later in the week. Since this patch has to be pushed on Tuesday (likely the day you are reading this) I don't want to add any risk due to even small changes. The change will be one place in each of: - jdk/make/src/GensrcMisc.gmk - jdk/make/CompileJavaClasses.gmk - jdk/make/CreateJars.gmk Pete p.s. Apparently there was a cockpit error on my part on my first attempt at the open only build (which should have failed but didn't). After JPRT found the problem I was able to reproduce the issue locally and now both local and JPRT open builds are OK. On 3/30/15 2:05 PM, Se?n Coffey wrote: > Pete, > > I'm not a reviewer for this but I notice that some files have the > incorrect copyright header (PROPRIETARY/CONFIDENTIAL) - Please correct > that before any push. I think it's also safer if you run both an > OpenJDK build and a closed/open src build through JPRT. It'll handle > both. > > regards, > Sean. > > On 30/03/2015 19:39, Pete Brunet wrote: >> p.s. The 7 JPRT builds ran OK: 2 win, 2 linux, 2 solaris, 1 mac >> >> On 3/30/15 11:10 AM, Pete Brunet wrote: >>> Erik, I made the changes that you pointed out. Builds worked OK for >>> local closed and open. I also started a JPRT build for all >>> platforms. That's 7 targets and since that will take a while I wanted >>> to publish the new patch before you left for the day: >>> >>> http://cr.openjdk.java.net/~ptbrunet/JDK-8076182/webrev.01/ >>> >>> Pete >>> >>> On 3/30/15 2:43 AM, Erik Joelsson wrote: >>>> Hello, >>>> >>>> In jdk/make/CompileJavaClasses.gmk: >>>> >>>> You removed >>>> $(JDK_TOPDIR)/src/closed/$(OPENJDK_TARGET_OS_API_DIR)/classes, but >>>> for linux and solaris, that source dir is still needed. Is it giving >>>> you an error when the directory doesn't exist? In that case you will >>>> need to make it conditional on platform. >>>> >>>> jdk/make/CompileLaunchers.gmk: >>>> >>>> If jabswitch is open source, then it shouldn't be built under the >>>> condition "ifndef OPENJDK". >>>> >>>> jdk/make/CopyFiles.gmk: >>>> >>>> ifndef OPENJDK >>>> >>>> jdk/make/gensrc/GensrcMisc.gmk: >>>> >>>> ifndef OPENJDK >>>> >>>> jdk/make/lib/PlatformLibraries.gmk: >>>> >>>> ifndef OPENJDK >>>> >>>> >>>> I would recommend getting an open only forest, apply your patch and >>>> verify that you get everything built properly. >>>> >>>> /Erik >>>> >>>> On 2015-03-28 01:30, Pete Brunet wrote: >>>>> Please review and approve a backport of JDK-8055831 to open source >>>>> the >>>>> Java Access Bridge. The original patch has been pushed to 9 and the >>>>> backport now needs to be pushed to 8u. >>>>> >>>>> Due to the significant directory structure changes in 9 due to >>>>> modularization I could not simply apply the patch. However the >>>>> effect >>>>> is functionally the same. Here are the changes: >>>>> >>>>> - moved the JAB files from closed to open >>>>> - change the copyright headers to open style >>>>> - adjust the make files accordingly >>>>> - create javadoc for com.sun.java.accessibility.util; creates it in >>>>> docs/jre/api/accessibility/jaccess/spec >>>>> - removed extraneous unused files; these are tools inadvertently >>>>> picked >>>>> up in 7u6 when the JAB was first integrated and which will be >>>>> added later >>>>> - removed some dead code >>>>> - cleaned up comments/documentation >>>>> - cleaned up some code indentation issues >>>>> - in package-private EventQueueMonitor.addTopLevelWindow the return >>>>> value was not used so I changed the return from boolean to void >>>>> >>>>> The open part of the patch is here: >>>>> http://cr.openjdk.java.net/~ptbrunet/JDK-8076182/webrev.00/ >>>>> >>>>> (I see I have five copyright dates to fix in jdk/make.) >>>>> >>>>> There is a closed part but it's just the removal of the source files >>>>> from the closed repository. >>>>> >>>>> The patch has been through a successful 32/64 bit JPRT build and >>>>> subsequent testing on my machine >>>>> >>>>> This patch needs to be pushed by Tuesday so I'll appreciate your >>>>> review. >>>>> >>>>> Thanks, Pete > From fredrik.arvidsson at oracle.com Tue Mar 31 08:28:51 2015 From: fredrik.arvidsson at oracle.com (Fredrik Arvidsson) Date: Tue, 31 Mar 2015 10:28:51 +0200 Subject: [8u60] Request for approval for CR 8076154 - com/sun/jdi/InstanceFilter.java failing due to missing MethodEntryRequest calls Message-ID: <551A5AC3.4060505@oracle.com> Please approve the back-port of the following test-bug from 9 to 8u60 (8u-dev). Bug: https://bugs.openjdk.java.net/browse/JDK-8076154 Review: http://cr.openjdk.java.net/~farvidsson/8076154/webrev.00/ Thanks /Fredrik From sean.coffey at oracle.com Tue Mar 31 08:46:39 2015 From: sean.coffey at oracle.com (=?UTF-8?B?U2XDoW4gQ29mZmV5?=) Date: Tue, 31 Mar 2015 09:46:39 +0100 Subject: [8u60] Request for approval for CR 8076154 - com/sun/jdi/InstanceFilter.java failing due to missing MethodEntryRequest calls In-Reply-To: <551A5AC3.4060505@oracle.com> References: <551A5AC3.4060505@oracle.com> Message-ID: <551A5EEF.1080702@oracle.com> Fredrik, Are you planning to fix this in JDK 9 first ? http://openjdk.java.net/projects/jdk8u/groundrules.html (# 1) It looks like you can mark the bug with a testbug label also. Do you have a link to the public code review thread ? If you're planning on pushing this into the hotspot team forest[1], you don't need an approval. Alejandro will handle that detail when a bulk push approval request is submitted. regards, Sean. [1] http://hg.openjdk.java.net/jdk8u/hs-dev/ On 31/03/2015 09:28, Fredrik Arvidsson wrote: > Please approve the back-port of the following test-bug from 9 to 8u60 > (8u-dev). > > Bug: > https://bugs.openjdk.java.net/browse/JDK-8076154 > > Review: > http://cr.openjdk.java.net/~farvidsson/8076154/webrev.00/ > > Thanks > /Fredrik From sean.coffey at oracle.com Tue Mar 31 09:50:31 2015 From: sean.coffey at oracle.com (=?UTF-8?B?U2XDoW4gQ29mZmV5?=) Date: Tue, 31 Mar 2015 10:50:31 +0100 Subject: RfR and RfA: Backport of JDK-8055831 Open Source Java Access Bridge - JDK-8076182 In-Reply-To: <551A12C6.70603@oracle.com> References: <5515F626.30808@oracle.com> <5518FEBB.7020103@oracle.com> <55197565.2090706@oracle.com> <5519985F.9030907@oracle.com> <55199E70.1000202@oracle.com> <551A12C6.70603@oracle.com> Message-ID: <551A6DE7.4080807@oracle.com> It looks like the bug ID of interest here is JDK-8076182. I've added that to subject line. You can consider this approved for jdk8u-dev once you get code review from an engineer in the build team. You should also add an appropriate noreg- label[1] to the bug report. regards, Sean. [1] http://openjdk.java.net/guide/changePlanning.html#noreg On 31/03/2015 04:21, Pete Brunet wrote: > Thanks Sean, The JPRT open build pointed out some problems which I have > resolved but need Erik's review of those changes which are here: > http://cr.openjdk.java.net/~ptbrunet/JDK-8076182/webrev.02/ > > Alexander Zvegintsev reviewed the source so once Erik finishes his > review hopefully you'll be able to approve it for push into 8u-dev. > > Erik, There are some places where I changed ifndef OpenJDK to ifdef > OpenJDK. I didn't think of it until too late in the evening, but I > probably should have just removed that. If so, I can do that later in > the week. Since this patch has to be pushed on Tuesday (likely the day > you are reading this) I don't want to add any risk due to even small > changes. The change will be one place in each of: > - jdk/make/src/GensrcMisc.gmk > - jdk/make/CompileJavaClasses.gmk > - jdk/make/CreateJars.gmk > > Pete > > p.s. Apparently there was a cockpit error on my part on my first attempt > at the open only build (which should have failed but didn't). After > JPRT found the problem I was able to reproduce the issue locally and now > both local and JPRT open builds are OK. > > On 3/30/15 2:05 PM, Se?n Coffey wrote: >> Pete, >> >> I'm not a reviewer for this but I notice that some files have the >> incorrect copyright header (PROPRIETARY/CONFIDENTIAL) - Please correct >> that before any push. I think it's also safer if you run both an >> OpenJDK build and a closed/open src build through JPRT. It'll handle >> both. >> >> regards, >> Sean. >> >> On 30/03/2015 19:39, Pete Brunet wrote: >>> p.s. The 7 JPRT builds ran OK: 2 win, 2 linux, 2 solaris, 1 mac >>> >>> On 3/30/15 11:10 AM, Pete Brunet wrote: >>>> Erik, I made the changes that you pointed out. Builds worked OK for >>>> local closed and open. I also started a JPRT build for all >>>> platforms. That's 7 targets and since that will take a while I wanted >>>> to publish the new patch before you left for the day: >>>> >>>> http://cr.openjdk.java.net/~ptbrunet/JDK-8076182/webrev.01/ >>>> >>>> Pete >>>> >>>> On 3/30/15 2:43 AM, Erik Joelsson wrote: >>>>> Hello, >>>>> >>>>> In jdk/make/CompileJavaClasses.gmk: >>>>> >>>>> You removed >>>>> $(JDK_TOPDIR)/src/closed/$(OPENJDK_TARGET_OS_API_DIR)/classes, but >>>>> for linux and solaris, that source dir is still needed. Is it giving >>>>> you an error when the directory doesn't exist? In that case you will >>>>> need to make it conditional on platform. >>>>> >>>>> jdk/make/CompileLaunchers.gmk: >>>>> >>>>> If jabswitch is open source, then it shouldn't be built under the >>>>> condition "ifndef OPENJDK". >>>>> >>>>> jdk/make/CopyFiles.gmk: >>>>> >>>>> ifndef OPENJDK >>>>> >>>>> jdk/make/gensrc/GensrcMisc.gmk: >>>>> >>>>> ifndef OPENJDK >>>>> >>>>> jdk/make/lib/PlatformLibraries.gmk: >>>>> >>>>> ifndef OPENJDK >>>>> >>>>> >>>>> I would recommend getting an open only forest, apply your patch and >>>>> verify that you get everything built properly. >>>>> >>>>> /Erik >>>>> >>>>> On 2015-03-28 01:30, Pete Brunet wrote: >>>>>> Please review and approve a backport of JDK-8055831 to open source >>>>>> the >>>>>> Java Access Bridge. The original patch has been pushed to 9 and the >>>>>> backport now needs to be pushed to 8u. >>>>>> >>>>>> Due to the significant directory structure changes in 9 due to >>>>>> modularization I could not simply apply the patch. However the >>>>>> effect >>>>>> is functionally the same. Here are the changes: >>>>>> >>>>>> - moved the JAB files from closed to open >>>>>> - change the copyright headers to open style >>>>>> - adjust the make files accordingly >>>>>> - create javadoc for com.sun.java.accessibility.util; creates it in >>>>>> docs/jre/api/accessibility/jaccess/spec >>>>>> - removed extraneous unused files; these are tools inadvertently >>>>>> picked >>>>>> up in 7u6 when the JAB was first integrated and which will be >>>>>> added later >>>>>> - removed some dead code >>>>>> - cleaned up comments/documentation >>>>>> - cleaned up some code indentation issues >>>>>> - in package-private EventQueueMonitor.addTopLevelWindow the return >>>>>> value was not used so I changed the return from boolean to void >>>>>> >>>>>> The open part of the patch is here: >>>>>> http://cr.openjdk.java.net/~ptbrunet/JDK-8076182/webrev.00/ >>>>>> >>>>>> (I see I have five copyright dates to fix in jdk/make.) >>>>>> >>>>>> There is a closed part but it's just the removal of the source files >>>>>> from the closed repository. >>>>>> >>>>>> The patch has been through a successful 32/64 bit JPRT build and >>>>>> subsequent testing on my machine >>>>>> >>>>>> This patch needs to be pushed by Tuesday so I'll appreciate your >>>>>> review. >>>>>> >>>>>> Thanks, Pete From erik.joelsson at oracle.com Tue Mar 31 10:45:53 2015 From: erik.joelsson at oracle.com (Erik Joelsson) Date: Tue, 31 Mar 2015 12:45:53 +0200 Subject: RfR and RfA: Backport of JDK-8055831 Open Source Java Access Bridge In-Reply-To: <551A12C6.70603@oracle.com> References: <5515F626.30808@oracle.com> <5518FEBB.7020103@oracle.com> <55197565.2090706@oracle.com> <5519985F.9030907@oracle.com> <55199E70.1000202@oracle.com> <551A12C6.70603@oracle.com> Message-ID: <551A7AE1.4070107@oracle.com> Hello Pete, The webrev.02 linked below is broken so I will not approve it. The "ifdef OPENJDK" means that those parts of the build will not get built when closed source is available, i.e. for Oracle builds. There are also formatting issues that, while not critical, I would normally not let through either. Since this seems urgent to you, I have prepared a webrev of the makefiles, corrected the way I would like them. I have built this on windows both 32 and 64 bit, both open and closed and verified that all accessbridge related build artifacts appear in all of them. http://cr.openjdk.java.net/~erikj/8076182/webrev.01/ I have started two JPRT jobs to verify nothing broke on the other platforms. /Erik On 2015-03-31 05:21, Pete Brunet wrote: > Thanks Sean, The JPRT open build pointed out some problems which I have > resolved but need Erik's review of those changes which are here: > http://cr.openjdk.java.net/~ptbrunet/JDK-8076182/webrev.02/ > > Alexander Zvegintsev reviewed the source so once Erik finishes his > review hopefully you'll be able to approve it for push into 8u-dev. > > Erik, There are some places where I changed ifndef OpenJDK to ifdef > OpenJDK. I didn't think of it until too late in the evening, but I > probably should have just removed that. If so, I can do that later in > the week. Since this patch has to be pushed on Tuesday (likely the day > you are reading this) I don't want to add any risk due to even small > changes. The change will be one place in each of: > - jdk/make/src/GensrcMisc.gmk > - jdk/make/CompileJavaClasses.gmk > - jdk/make/CreateJars.gmk > > Pete > > p.s. Apparently there was a cockpit error on my part on my first attempt > at the open only build (which should have failed but didn't). After > JPRT found the problem I was able to reproduce the issue locally and now > both local and JPRT open builds are OK. > > On 3/30/15 2:05 PM, Se?n Coffey wrote: >> Pete, >> >> I'm not a reviewer for this but I notice that some files have the >> incorrect copyright header (PROPRIETARY/CONFIDENTIAL) - Please correct >> that before any push. I think it's also safer if you run both an >> OpenJDK build and a closed/open src build through JPRT. It'll handle >> both. >> >> regards, >> Sean. >> >> On 30/03/2015 19:39, Pete Brunet wrote: >>> p.s. The 7 JPRT builds ran OK: 2 win, 2 linux, 2 solaris, 1 mac >>> >>> On 3/30/15 11:10 AM, Pete Brunet wrote: >>>> Erik, I made the changes that you pointed out. Builds worked OK for >>>> local closed and open. I also started a JPRT build for all >>>> platforms. That's 7 targets and since that will take a while I wanted >>>> to publish the new patch before you left for the day: >>>> >>>> http://cr.openjdk.java.net/~ptbrunet/JDK-8076182/webrev.01/ >>>> >>>> Pete >>>> >>>> On 3/30/15 2:43 AM, Erik Joelsson wrote: >>>>> Hello, >>>>> >>>>> In jdk/make/CompileJavaClasses.gmk: >>>>> >>>>> You removed >>>>> $(JDK_TOPDIR)/src/closed/$(OPENJDK_TARGET_OS_API_DIR)/classes, but >>>>> for linux and solaris, that source dir is still needed. Is it giving >>>>> you an error when the directory doesn't exist? In that case you will >>>>> need to make it conditional on platform. >>>>> >>>>> jdk/make/CompileLaunchers.gmk: >>>>> >>>>> If jabswitch is open source, then it shouldn't be built under the >>>>> condition "ifndef OPENJDK". >>>>> >>>>> jdk/make/CopyFiles.gmk: >>>>> >>>>> ifndef OPENJDK >>>>> >>>>> jdk/make/gensrc/GensrcMisc.gmk: >>>>> >>>>> ifndef OPENJDK >>>>> >>>>> jdk/make/lib/PlatformLibraries.gmk: >>>>> >>>>> ifndef OPENJDK >>>>> >>>>> >>>>> I would recommend getting an open only forest, apply your patch and >>>>> verify that you get everything built properly. >>>>> >>>>> /Erik >>>>> >>>>> On 2015-03-28 01:30, Pete Brunet wrote: >>>>>> Please review and approve a backport of JDK-8055831 to open source >>>>>> the >>>>>> Java Access Bridge. The original patch has been pushed to 9 and the >>>>>> backport now needs to be pushed to 8u. >>>>>> >>>>>> Due to the significant directory structure changes in 9 due to >>>>>> modularization I could not simply apply the patch. However the >>>>>> effect >>>>>> is functionally the same. Here are the changes: >>>>>> >>>>>> - moved the JAB files from closed to open >>>>>> - change the copyright headers to open style >>>>>> - adjust the make files accordingly >>>>>> - create javadoc for com.sun.java.accessibility.util; creates it in >>>>>> docs/jre/api/accessibility/jaccess/spec >>>>>> - removed extraneous unused files; these are tools inadvertently >>>>>> picked >>>>>> up in 7u6 when the JAB was first integrated and which will be >>>>>> added later >>>>>> - removed some dead code >>>>>> - cleaned up comments/documentation >>>>>> - cleaned up some code indentation issues >>>>>> - in package-private EventQueueMonitor.addTopLevelWindow the return >>>>>> value was not used so I changed the return from boolean to void >>>>>> >>>>>> The open part of the patch is here: >>>>>> http://cr.openjdk.java.net/~ptbrunet/JDK-8076182/webrev.00/ >>>>>> >>>>>> (I see I have five copyright dates to fix in jdk/make.) >>>>>> >>>>>> There is a closed part but it's just the removal of the source files >>>>>> from the closed repository. >>>>>> >>>>>> The patch has been through a successful 32/64 bit JPRT build and >>>>>> subsequent testing on my machine >>>>>> >>>>>> This patch needs to be pushed by Tuesday so I'll appreciate your >>>>>> review. >>>>>> >>>>>> Thanks, Pete From peter.brunet at oracle.com Tue Mar 31 11:12:10 2015 From: peter.brunet at oracle.com (Pete Brunet) Date: Tue, 31 Mar 2015 06:12:10 -0500 Subject: RfR and RfA: Backport of JDK-8055831 Open Source Java Access Bridge In-Reply-To: <551A7AE1.4070107@oracle.com> References: <5515F626.30808@oracle.com> <5518FEBB.7020103@oracle.com> <55197565.2090706@oracle.com> <5519985F.9030907@oracle.com> <55199E70.1000202@oracle.com> <551A12C6.70603@oracle.com> <551A7AE1.4070107@oracle.com> Message-ID: <551A810A.9040400@oracle.com> Hi Erik, Thank you for the review and assistance. -Pete On 3/31/15 5:45 AM, Erik Joelsson wrote: > Hello Pete, > > The webrev.02 linked below is broken so I will not approve it. The > "ifdef OPENJDK" means that those parts of the build will not get built > when closed source is available, i.e. for Oracle builds. There are > also formatting issues that, while not critical, I would normally not > let through either. > > Since this seems urgent to you, I have prepared a webrev of the > makefiles, corrected the way I would like them. I have built this on > windows both 32 and 64 bit, both open and closed and verified that all > accessbridge related build artifacts appear in all of them. > > http://cr.openjdk.java.net/~erikj/8076182/webrev.01/ > > I have started two JPRT jobs to verify nothing broke on the other > platforms. > > /Erik > > On 2015-03-31 05:21, Pete Brunet wrote: >> Thanks Sean, The JPRT open build pointed out some problems which I have >> resolved but need Erik's review of those changes which are here: >> http://cr.openjdk.java.net/~ptbrunet/JDK-8076182/webrev.02/ >> >> Alexander Zvegintsev reviewed the source so once Erik finishes his >> review hopefully you'll be able to approve it for push into 8u-dev. >> >> Erik, There are some places where I changed ifndef OpenJDK to ifdef >> OpenJDK. I didn't think of it until too late in the evening, but I >> probably should have just removed that. If so, I can do that later in >> the week. Since this patch has to be pushed on Tuesday (likely the day >> you are reading this) I don't want to add any risk due to even small >> changes. The change will be one place in each of: >> - jdk/make/src/GensrcMisc.gmk >> - jdk/make/CompileJavaClasses.gmk >> - jdk/make/CreateJars.gmk >> >> Pete >> >> p.s. Apparently there was a cockpit error on my part on my first attempt >> at the open only build (which should have failed but didn't). After >> JPRT found the problem I was able to reproduce the issue locally and now >> both local and JPRT open builds are OK. >> >> On 3/30/15 2:05 PM, Se?n Coffey wrote: >>> Pete, >>> >>> I'm not a reviewer for this but I notice that some files have the >>> incorrect copyright header (PROPRIETARY/CONFIDENTIAL) - Please correct >>> that before any push. I think it's also safer if you run both an >>> OpenJDK build and a closed/open src build through JPRT. It'll handle >>> both. >>> >>> regards, >>> Sean. >>> >>> On 30/03/2015 19:39, Pete Brunet wrote: >>>> p.s. The 7 JPRT builds ran OK: 2 win, 2 linux, 2 solaris, 1 mac >>>> >>>> On 3/30/15 11:10 AM, Pete Brunet wrote: >>>>> Erik, I made the changes that you pointed out. Builds worked OK for >>>>> local closed and open. I also started a JPRT build for all >>>>> platforms. That's 7 targets and since that will take a while I >>>>> wanted >>>>> to publish the new patch before you left for the day: >>>>> >>>>> http://cr.openjdk.java.net/~ptbrunet/JDK-8076182/webrev.01/ >>>>> >>>>> Pete >>>>> >>>>> On 3/30/15 2:43 AM, Erik Joelsson wrote: >>>>>> Hello, >>>>>> >>>>>> In jdk/make/CompileJavaClasses.gmk: >>>>>> >>>>>> You removed >>>>>> $(JDK_TOPDIR)/src/closed/$(OPENJDK_TARGET_OS_API_DIR)/classes, but >>>>>> for linux and solaris, that source dir is still needed. Is it giving >>>>>> you an error when the directory doesn't exist? In that case you will >>>>>> need to make it conditional on platform. >>>>>> >>>>>> jdk/make/CompileLaunchers.gmk: >>>>>> >>>>>> If jabswitch is open source, then it shouldn't be built under the >>>>>> condition "ifndef OPENJDK". >>>>>> >>>>>> jdk/make/CopyFiles.gmk: >>>>>> >>>>>> ifndef OPENJDK >>>>>> >>>>>> jdk/make/gensrc/GensrcMisc.gmk: >>>>>> >>>>>> ifndef OPENJDK >>>>>> >>>>>> jdk/make/lib/PlatformLibraries.gmk: >>>>>> >>>>>> ifndef OPENJDK >>>>>> >>>>>> >>>>>> I would recommend getting an open only forest, apply your patch and >>>>>> verify that you get everything built properly. >>>>>> >>>>>> /Erik >>>>>> >>>>>> On 2015-03-28 01:30, Pete Brunet wrote: >>>>>>> Please review and approve a backport of JDK-8055831 to open source >>>>>>> the >>>>>>> Java Access Bridge. The original patch has been pushed to 9 and >>>>>>> the >>>>>>> backport now needs to be pushed to 8u. >>>>>>> >>>>>>> Due to the significant directory structure changes in 9 due to >>>>>>> modularization I could not simply apply the patch. However the >>>>>>> effect >>>>>>> is functionally the same. Here are the changes: >>>>>>> >>>>>>> - moved the JAB files from closed to open >>>>>>> - change the copyright headers to open style >>>>>>> - adjust the make files accordingly >>>>>>> - create javadoc for com.sun.java.accessibility.util; creates it in >>>>>>> docs/jre/api/accessibility/jaccess/spec >>>>>>> - removed extraneous unused files; these are tools inadvertently >>>>>>> picked >>>>>>> up in 7u6 when the JAB was first integrated and which will be >>>>>>> added later >>>>>>> - removed some dead code >>>>>>> - cleaned up comments/documentation >>>>>>> - cleaned up some code indentation issues >>>>>>> - in package-private EventQueueMonitor.addTopLevelWindow the return >>>>>>> value was not used so I changed the return from boolean to void >>>>>>> >>>>>>> The open part of the patch is here: >>>>>>> http://cr.openjdk.java.net/~ptbrunet/JDK-8076182/webrev.00/ >>>>>>> >>>>>>> (I see I have five copyright dates to fix in jdk/make.) >>>>>>> >>>>>>> There is a closed part but it's just the removal of the source >>>>>>> files >>>>>>> from the closed repository. >>>>>>> >>>>>>> The patch has been through a successful 32/64 bit JPRT build and >>>>>>> subsequent testing on my machine >>>>>>> >>>>>>> This patch needs to be pushed by Tuesday so I'll appreciate your >>>>>>> review. >>>>>>> >>>>>>> Thanks, Pete > From fredrik.arvidsson at oracle.com Tue Mar 31 11:58:32 2015 From: fredrik.arvidsson at oracle.com (Fredrik Arvidsson) Date: Tue, 31 Mar 2015 13:58:32 +0200 Subject: [8u60] Request for approval for CR 8076154 - com/sun/jdi/InstanceFilter.java failing due to missing MethodEntryRequest calls In-Reply-To: <551A5EEF.1080702@oracle.com> References: <551A5AC3.4060505@oracle.com> <551A5EEF.1080702@oracle.com> Message-ID: <551A8BE8.7060503@oracle.com> Hi I added 'testbug' label on the bug. Here is the change in 9: http://hg.openjdk.java.net/jdk9/hs-rt/jdk/rev/ce259f9be831 Since this really isn't a hotspot bug I would like to get an approval for pushing it directly to jdk8u-dev. Cheers /Fredrik On 2015-03-31 10:46, Se?n Coffey wrote: > Fredrik, > > Are you planning to fix this in JDK 9 first ? > http://openjdk.java.net/projects/jdk8u/groundrules.html (# 1) > > It looks like you can mark the bug with a testbug label also. Do you > have a link to the public code review thread ? If you're planning on > pushing this into the hotspot team forest[1], you don't need an > approval. Alejandro will handle that detail when a bulk push approval > request is submitted. > > regards, > Sean. > > [1] http://hg.openjdk.java.net/jdk8u/hs-dev/ > > On 31/03/2015 09:28, Fredrik Arvidsson wrote: >> Please approve the back-port of the following test-bug from 9 to 8u60 >> (8u-dev). >> >> Bug: >> https://bugs.openjdk.java.net/browse/JDK-8076154 >> >> Review: >> http://cr.openjdk.java.net/~farvidsson/8076154/webrev.00/ >> >> Thanks >> /Fredrik > From sean.coffey at oracle.com Tue Mar 31 13:06:36 2015 From: sean.coffey at oracle.com (=?UTF-8?B?U2XDoW4gQ29mZmV5?=) Date: Tue, 31 Mar 2015 14:06:36 +0100 Subject: [8u60] Request for approval for CR 8076154 - com/sun/jdi/InstanceFilter.java failing due to missing MethodEntryRequest calls In-Reply-To: <551A8BE8.7060503@oracle.com> References: <551A5AC3.4060505@oracle.com> <551A5EEF.1080702@oracle.com> <551A8BE8.7060503@oracle.com> Message-ID: <551A9BDC.9010602@oracle.com> JDK 9 review thread : http://mail.openjdk.java.net/pipermail/serviceability-dev/2015-March/016986.html Approved. Thanks for adding label. regards, Sean. On 31/03/2015 12:58, Fredrik Arvidsson wrote: > Hi > > I added 'testbug' label on the bug. > Here is the change in 9: > http://hg.openjdk.java.net/jdk9/hs-rt/jdk/rev/ce259f9be831 > > Since this really isn't a hotspot bug I would like to get an approval > for pushing it directly to jdk8u-dev. > > Cheers > /Fredrik > > On 2015-03-31 10:46, Se?n Coffey wrote: >> Fredrik, >> >> Are you planning to fix this in JDK 9 first ? >> http://openjdk.java.net/projects/jdk8u/groundrules.html (# 1) >> >> It looks like you can mark the bug with a testbug label also. Do you >> have a link to the public code review thread ? If you're planning on >> pushing this into the hotspot team forest[1], you don't need an >> approval. Alejandro will handle that detail when a bulk push approval >> request is submitted. >> >> regards, >> Sean. >> >> [1] http://hg.openjdk.java.net/jdk8u/hs-dev/ >> >> On 31/03/2015 09:28, Fredrik Arvidsson wrote: >>> Please approve the back-port of the following test-bug from 9 to >>> 8u60 (8u-dev). >>> >>> Bug: >>> https://bugs.openjdk.java.net/browse/JDK-8076154 >>> >>> Review: >>> http://cr.openjdk.java.net/~farvidsson/8076154/webrev.00/ >>> >>> Thanks >>> /Fredrik >> > From alejandro.murillo at oracle.com Tue Mar 31 19:26:41 2015 From: alejandro.murillo at oracle.com (Alejandro E Murillo) Date: Tue, 31 Mar 2015 13:26:41 -0600 Subject: jdk8u60-b09: HotSpot Message-ID: <551AF4F1.9040006@oracle.com> hs25.60-b09 has been integrated into jdk8u60-b09. http://hg.openjdk.java.net/jdk8u/jdk8u/rev/da6d0717c74a http://hg.openjdk.java.net/jdk8u/jdk8u/corba/rev/e8af97f98cad http://hg.openjdk.java.net/jdk8u/jdk8u/hotspot/rev/f1058b5c6294 http://hg.openjdk.java.net/jdk8u/jdk8u/jaxp/rev/566ff4ee50bb http://hg.openjdk.java.net/jdk8u/jdk8u/jaxws/rev/fbb7b2d1321f http://hg.openjdk.java.net/jdk8u/jdk8u/jdk/rev/fc3f69854e7d http://hg.openjdk.java.net/jdk8u/jdk8u/langtools/rev/76adee5ad278 http://hg.openjdk.java.net/jdk8u/jdk8u/nashorn/rev/1f73439a45bf Component : VM Status : 2 major failures, 2 minor failures Date : 03/31/2015 at 18:00 MSK Tested By : STT_VM Cost(total man-days): 1 Workspace : 2015-03-27-174334.amurillo.hs25-60-b09-snapshot Bundles : 2015-03-27-174334.amurillo.hs25-60-b09-snapshot Platforms : Others Tests :/net/sqenfs-1.sfbay/export1/comp/vm/testbase/ Log : link Browsers : NA Patches : NA Number of Tests Executed : 431331 passed tests, 4245 failed tests (many new failures) Bug verification status: ====================================== Tested, Pass: Tested, Pass (partial fixes): Tested, Fail: Untested bug fixes: 7127066: Class verifier accepts an invalid class file 8013942: JSR 292: assert(type() == T_OBJECT) failed: type check 8027962: Per-phase timing measurements for strong roots processing 8051837: Remove temporary G1UseParallelRSetUpdating and G1UseParallelRSetScanning flags 8053998: Hot card cache flush chunk size too coarse grained 8055416: Several vm/gc/heap/summary "After GC" events emitted for the same GC ID 8058801: G1TraceReclaimDeadHumongousObjectsAtYoungGC only prints humongous object liveness output when there is at least one candidate humongous object 8060025: Object copy time regressions after JDK-8031323 and JDK-8057536 8061630: G1 iterates over JNIHandles two times 8064473: Improved handling of age during object copy in G1 8065358: Refactor G1s usage of save_marks and reduce related races 8067655: Clean up G1 remembered set oop iteration 8067923: AIX: link libjvm.so with -bernotok to detect missing symbols at build time and suppress warning 1540-1639 8069273: Decrease Hot Card Cache Lock contention 8069760: When iterating over a card, G1 often iterates over much more references than are contained in the card 8072740: move closed jvm.cfg files out of open repo 8074037: Refactor the G1GCPhaseTime logging to make it easier to add new phases 8075118: JVM stuck in infinite loop during verification 8075210: Refactor strong root processing in order to allow G1 to evolve separately from GenCollectedHeap 8075615: new hotspot build - hs25.60-b09 8075858: AIX: clean-up HotSpot make files New bugs filed: Bugs in PIT build: 8068036: assert(is_available(index)) failed in G1 cset 8076295: SIGSEGV G1ParGCAllocator::create_allocator 8076300: C2: assert(i < _max) failed: oob: i=0, _max=0 8076301: C1: Error: ShouldNotReachHere() LIR_OpVisitState::visit Bugs in earlier promoted build: Number of PIT requested: 1 Integration target J2SE build number: jdk8u60-b09 Issues and Notes: This is PIT for HS25.60-b09 for jdk8u60-b09. SQE is OK integrating as long as 8068036 & 8076295 are fixed by the next integration. Fix for 8068036 has been checked in and the one for 8076295 will be in by the end of the week. -- Alejandro