Re: [MERGE FOR REVIEW] HotSpot 23.25-b01 Merge

2013-09-12 Thread Andrew Hughes
- Original Message - > Hi, > > In case it was not clear, please go ahead with the push. > added 1587 changesets with 6604 changes to 1453 files :-) > On 09/12/2013 04:01 PM, Andrew Hughes wrote: > >> One concern, though (not that it needs to be resolved right now, but I > >> figured

Re: [MERGE FOR REVIEW] HotSpot 23.25-b01 Merge

2013-09-12 Thread Andrew Hughes
- Original Message - > Hi, > > In case it was not clear, please go ahead with the push. > > On 09/12/2013 04:01 PM, Andrew Hughes wrote: > >> One concern, though (not that it needs to be resolved right now, but I > >> figured I would point it out anyway): > > (snip) > > > Maybe, but e

Re: [MERGE FOR REVIEW] HotSpot 23.25-b01 Merge

2013-09-12 Thread Andrew Hughes
- Original Message - > On 09/12/2013 09:20 PM, Andrew Hughes wrote: > > > > > > - Original Message - > >> On 09/12/2013 10:06 AM, Andrew Hughes wrote: > >>> Enjoy: > >>> > >>> http://cr.openjdk.java.net/~andrew/jdk6-hs23-merge/ > >> > >> I tried to apply the patch linked in the webr

Re: [MERGE FOR REVIEW] HotSpot 23.25-b01 Merge

2013-09-12 Thread Andrew Hughes
- Original Message - > On 09/12/2013 10:06 AM, Andrew Hughes wrote: > > Enjoy: > > > > http://cr.openjdk.java.net/~andrew/jdk6-hs23-merge/ > > > > Differences that remain between this and jdk7u25-b15 [0]: > > Thanks for the patch and explaining the differences. All the changes > looks se

Re: [MERGE FOR REVIEW] HotSpot 23.25-b01 Merge

2013-09-12 Thread Omair Majid
Hi, In case it was not clear, please go ahead with the push. On 09/12/2013 04:01 PM, Andrew Hughes wrote: >> One concern, though (not that it needs to be resolved right now, but I >> figured I would point it out anyway): (snip) > Maybe, but either way, that's a patch to apply on top, not someth

Re: [MERGE FOR REVIEW] HotSpot 23.25-b01 Merge

2013-09-12 Thread Alex Kasko
On 09/12/2013 09:20 PM, Andrew Hughes wrote: - Original Message - On 09/12/2013 10:06 AM, Andrew Hughes wrote: Enjoy: http://cr.openjdk.java.net/~andrew/jdk6-hs23-merge/ I tried to apply the patch linked in the webrev, but it fails to apply. The patch contains changes like [1] part

Re: [MERGE FOR REVIEW] HotSpot 23.25-b01 Merge

2013-09-12 Thread Omair Majid
On 09/12/2013 10:06 AM, Andrew Hughes wrote: > Enjoy: > > http://cr.openjdk.java.net/~andrew/jdk6-hs23-merge/ > > Differences that remain between this and jdk7u25-b15 [0]: Thanks for the patch and explaining the differences. All the changes looks sensible to me. > * src/os/linux/vm/os_linux.cpp

Re: [MERGE FOR REVIEW] HotSpot 23.25-b01 Merge

2013-09-12 Thread Andrew Hughes
- Original Message - > On 09/12/2013 10:06 AM, Andrew Hughes wrote: > > Enjoy: > > > > http://cr.openjdk.java.net/~andrew/jdk6-hs23-merge/ > > I tried to apply the patch linked in the webrev, but it fails to apply. > The patch contains changes like [1] part of which are already present

Re: [MERGE FOR REVIEW] HotSpot 23.25-b01 Merge

2013-09-12 Thread Omair Majid
On 09/12/2013 10:06 AM, Andrew Hughes wrote: > Enjoy: > > http://cr.openjdk.java.net/~andrew/jdk6-hs23-merge/ I tried to apply the patch linked in the webrev, but it fails to apply. The patch contains changes like [1] part of which are already present in jdk6/hotspot [2]. I am not sure if this is

[MERGE FOR REVIEW] HotSpot 23.25-b01 Merge

2013-09-12 Thread Andrew Hughes
Enjoy: http://cr.openjdk.java.net/~andrew/jdk6-hs23-merge/ Differences that remain between this and jdk7u25-b15 [0]: * Formatting correction in agent/test/jdi/sagtest.java * ^webrev/ in .hgignore * .jcheck/conf settings * hotspot/make/hotspot_version has JDK_MINOR_VER=6 * hotspot/make/jprt.prope