Re: Need reviewer and approval - jaxws drop bundle removal

2012-05-11 Thread Tim Bell
Hi Kelly This webrev folds the 2 source drops for jaxws back into the jdk7-dev/jaxws repository: http://cr.openjdk.java.net/~ohair/openjdk7u/jaxws-no-drop/webrev/ I've left out the final steps on pulling in the sources, makes too big of a webrev: sh ./make/scripts/update_src.sh hg

Need reviewer and approval - jaxws drop bundle removal

2012-05-11 Thread Kelly O'Hair
This webrev folds the 2 source drops for jaxws back into the jdk7-dev/jaxws repository: http://cr.openjdk.java.net/~ohair/openjdk7u/jaxws-no-drop/webrev/ I've left out the final steps on pulling in the sources, makes too big of a webrev: sh ./make/scripts/update_src.sh hg add src I ve

hg: jdk7u/jdk7u-dev/jaxws: 7168374: Upgrade of XML components to JAXB 2.2.4-2, SAAJ 1.3.16, JAXWS 2.2.4-1

2012-05-11 Thread kelly . ohair
Changeset: 376cbfcbf785 Author:ohair Date: 2012-05-11 18:48 -0700 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jaxws/rev/376cbfcbf785 7168374: Upgrade of XML components to JAXB 2.2.4-2, SAAJ 1.3.16, JAXWS 2.2.4-1 Summary: JAX-WS review by Martin Grebac Reviewed-by: tbell Contrib

Re: Need reviewer and approval

2012-05-11 Thread Tim Bell
Hi Kelly: 7168374: Upgrade of XML components to JAXB 2.2.4-2, SAAJ 1.3.16, JAXWS 2.2.4-1 http://cr.openjdk.java.net/~ohair/openjdk7u/jaxws224/webrev/ I plan on pushing it intohttp://hg.openjdk.java.net/jdk7u/jdk7u-dev/jaxws The JAX-WS team does not have OpenJDK usernames yet, so I need some

Need reviewer and approval

2012-05-11 Thread Kelly O'Hair
I'm trying to help the JAX-WS team integrate a new JAX-WS 2.2.4 source drop. 7168374: Upgrade of XML components to JAXB 2.2.4-2, SAAJ 1.3.16, JAXWS 2.2.4-1 http://cr.openjdk.java.net/~ohair/openjdk7u/jaxws224/webrev/ I plan on pushing it intohttp://hg.openjdk.java.net/jdk7u/jdk7u-dev/jaxws

[7u6] Request for approval for bulk integration of hs23.2-b01

2012-05-11 Thread Alejandro E Murillo
Requesting approval to integrate hs23.2-b01 into jdk7u6-b10. A webrev is available at: http://cr.openjdk.java.net/~amurillo/7u6/hs23.2-b01-jdk7u6-b10.webrev/ Pre-integration testing is in progress; the integration will proceed only after SQE has analyzed the results and approved. The fixes in

Re: JDK 7u4 is now GA

2012-05-11 Thread Edvard Wendelin
Hi, Source bundle is now available here: http://jdk7.java.net/source.html /Edvard On Apr 27, 2012, at 9:52 AM, Edvard Wendelin wrote: > Hi, > > I'd like to announce that JDK 7u4 is now generally available [1]. I would > like to thank all of you for the contributions you've made! OpenJDK 7u4

hg: jdk7u/jdk7u-dev/jdk: 7149320: Move sun.misc.VM.booted() to the end of System.initializeSystemClass()

2012-05-11 Thread mike . duigou
Changeset: 8004f3dd5c54 Author:mduigou Date: 2012-05-11 10:46 -0700 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/8004f3dd5c54 7149320: Move sun.misc.VM.booted() to the end of System.initializeSystemClass() Summary: Ensure that sun.misc.VM.booted() is the last action in S

New jdk7u maintainer : Sean Coffey

2012-05-11 Thread Dalibor Topic
I'm delegating the maintainer authority for jdk7u to Sean Coffey. That means that we now have four jdk7u forest maintainers again, as listed on the project page. Welcome, Sean! cheers, dalibor topic -- Oracle Dalibor Topic | Principal Product Manager Phone: +494089091214

Re: Review OSX universal mode patch

2012-05-11 Thread Henri Gomez
> But the OS X build system builds for multiple architectures fundamentally > differently than any other toolchain on any other OS. Yes and clearly it rocks ! > By doing multiple passes on the same file at the same time, the disk cache is > most effectively utilized. By simply tacking on additi

[7u6] request for approval, 7071826: Avoid benign race condition in initialization of UUID

2012-05-11 Thread Mike Duigou
Hi, This is a request to backport a JDK 8 fix into JDK 7u6: 7071826: Avoid benign race condition in initialization of UUID The fix has already been pushed to JDK 8: http://hg.openjdk.java.net/jdk8/tl-gate/jdk/rev/944676ef3c58 The code changes for jdk7u6 are identical to the ones in jdk8.

Re: Review OSX universal mode patch

2012-05-11 Thread Mike Swingler
On May 10, 2012, at 5:34 PM, David Holmes wrote: > On 11/05/2012 6:06 AM, Henri Gomez wrote: >> Happy to see subject back to technical. >> >> If you take a closer look to patch, you'll see universal is enforced as ARCH. > > My concerns with the patch is the necessity of introducing these platfor

Re: Review OSX universal mode patch

2012-05-11 Thread Henri Gomez
> For experimental support of non-mainstream BSD-ish platforms the bsd > porting project in OpenJDK would be my first choice. Why moving to BSD something which is purely OSX related ? Patches could be made safe enough to avoid conflicts with current mode, aka x86_64 arch.

hg: jdk7u/jdk7u-dev/jdk: 7167359: (tz) SEGV on solaris if TZ variable not set

2012-05-11 Thread sean . coffey
Changeset: 96bbb23bbaca Author:coffeys Date: 2012-05-11 11:52 +0100 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/96bbb23bbaca 7167359: (tz) SEGV on solaris if TZ variable not set Reviewed-by: okutsu ! src/solaris/native/java/util/TimeZone_md.c

Re: [7u6] Request for approval for CR 7167359 (tz) SEGV on solaris if TZ variable not set

2012-05-11 Thread Edvard Wendelin
Approved. On 05/11/2012 12:46 PM, Seán Coffey wrote: Looking to port this simple fix from 8 to 7u. Fix is identical to jdk 8 one. review thread : http://mail.openjdk.java.net/pipermail/i18n-dev/2012-May/000615.html jdk 8 changeset : http://hg.openjdk.java.net/jdk8/tl/jdk/rev/3e83229a3779 r

[7u6] Request for approval for CR 7167359 (tz) SEGV on solaris if TZ variable not set

2012-05-11 Thread Seán Coffey
Looking to port this simple fix from 8 to 7u. Fix is identical to jdk 8 one. review thread : http://mail.openjdk.java.net/pipermail/i18n-dev/2012-May/000615.html jdk 8 changeset : http://hg.openjdk.java.net/jdk8/tl/jdk/rev/3e83229a3779 regards, Sean.

Re: Review OSX universal mode patch

2012-05-11 Thread Dalibor Topic
On 5/11/12 10:04 AM, Henri Gomez wrote: > Dropping 32bits support in the future ? Near or long term future ? > > There is still SnowLeopard around and many may remains in the future, > we should try to preserve them, at least for the community supported > version. > > You mention fork, does it me

Re: Review OSX universal mode patch

2012-05-11 Thread Henri Gomez
> The patch looks fine to me. The value of "universal" is required to let the > build infrastructure pass both architectures to the compiler, and let the > compiler do the work of running twice, and then combining the results. Good news, in fact patches are back port of settings available in mac

Re: Review OSX universal mode patch

2012-05-11 Thread Henri Gomez
> Let's make it clear once and for all what is desired. I can put this into a > bug if it will help. > > -- A straight 'make' of OpenJDK on Mac OS X is 64-bit only. '-d32' and '-d64' > are ignored. > -- 'make all_xxx_universal' produces a universal build. (Some clarification > on the make target