Re: RFR: JDK-8001191: use -source 8 -target 8 when compiling the JDK

2012-11-07 Thread Erik Joelsson
Currently jaxp, jaxws and corba use explicit -source 7 -target 7 using the bootstrap javac in the old build. In build infra it seems this didn't get converted and these three repos are being compiled with the defaults from the bootstrap javac. The comparisons match up since since the defaults

Re: XS RFR: 8002040 Allow Full Debug Symbols when cross-compiling

2012-11-07 Thread Erik Joelsson
Looks good to me. /Erik On 2012-11-07 06:08, David Holmes wrote: webrev: http://cr.openjdk.java.net/~dholmes/8002040/webrev/ This is a simplified variant of the change just made to Hotspot. Instead of disabling FDS when cross-compiling we change the default location of objcopy, and just as

Review request: 8002365: build-infra: Build-infra fails on solaris 11.1 on sparc

2012-11-07 Thread Erik Joelsson
This is essentially the patch from Vincent to make build-infra work on Solaris 11.1 with some minor adjustments to keep the new and old build equal. http://cr.openjdk.java.net/~erikj/8002365/webrev.01/ http://cr.openjdk.java.net/%7Eerikj/8002365/webrev.01/ /Erik

Re: XS RFR: 8002040 Allow Full Debug Symbols when cross-compiling

2012-11-07 Thread Daniel D. Daugherty
On 11/6/12 10:08 PM, David Holmes wrote: webrev: http://cr.openjdk.java.net/~dholmes/8002040/webrev/ Thumbs up. make/common/Defs-linux.gmk No comments. This is a simplified variant of the change just made to Hotspot. Instead of disabling FDS when cross-compiling we change the default

Re: XS RFR: 8002040 Allow Full Debug Symbols when cross-compiling

2012-11-07 Thread Tim Bell
On 11/06/12 21:08, David Holmes wrote: webrev: http://cr.openjdk.java.net/~dholmes/8002040/webrev/ This is a simplified variant of the change just made to Hotspot. Instead of disabling FDS when cross-compiling we change the default location of objcopy, and just as with native compilation

Review Request: 8002028: build-infra: need no-hotspot partial build

2012-11-07 Thread Erik Joelsson
Adding support for importing the hotspot binaries from a prebuilt j2sdk image. Use configure option --with-import-hotspot=/path/to/j2sdkimage and the build will skip building hotspot. http://cr.openjdk.java.net/~erikj/8002028/webrev.jdk.01/

Re: Review request: 8002365: build-infra: Build-infra fails on solaris 11.1 on sparc

2012-11-07 Thread Tim Bell
On 11/07/12 06:38, Erik Joelsson wrote: This is essentially the patch from Vincent to make build-infra work on Solaris 11.1 with some minor adjustments to keep the new and old build equal. http://cr.openjdk.java.net/~erikj/8002365/webrev.01/

CFV: Nomination of Erik Joelsson to OpenJDK Build Group

2012-11-07 Thread Kelly O'Hair
I hereby nominate Erik Joelsson to Membership in the OpenJDK Build Group. Erik Joelsson (http://openjdk.java.net/census#erikj) is a Committer to the Build Infrastructure Project and author in the HotSpot Express and JDK 8 Projects. Erik is a member of the JDK Infrastructure team in Stockholm

Re: Review request: 8002365: build-infra: Build-infra fails on solaris 11.1 on sparc

2012-11-07 Thread Kelly O'Hair
Looks good to me. -kto On Nov 7, 2012, at 6:38 AM, Erik Joelsson wrote: This is essentially the patch from Vincent to make build-infra work on Solaris 11.1 with some minor adjustments to keep the new and old build equal. http://cr.openjdk.java.net/~erikj/8002365/webrev.01/

Working with common/autoconf/configure

2012-11-07 Thread David Chase
I'm working with a newly configured Linux box, trying the new build system, and I think I ran into some glitches (that I worked around). The box (uname -a) Linux bran 2.6.39-200.24.1.el6uek.x86_64 #1 SMP Sat Jun 23 02:39:07 EDT 2012 x86_64 x86_64 x86_64 GNU/Linux Before doing much work, I had

hg: jdk8/build/corba: 4 new changesets

2012-11-07 Thread david . katleman
Changeset: b450c07849ab Author:katleman Date: 2012-11-01 14:11 -0700 URL: http://hg.openjdk.java.net/jdk8/build/corba/rev/b450c07849ab Added tag jdk8-b63 for changeset 6ccbf67b68bf ! .hgtags Changeset: 643e7612cf6d Author:ohrstrom Date: 2012-10-29 14:10 +0100 URL:

hg: jdk8/build: 5 new changesets

2012-11-07 Thread david . katleman
Changeset: cababb9dfce7 Author:katleman Date: 2012-11-01 14:10 -0700 URL: http://hg.openjdk.java.net/jdk8/build/rev/cababb9dfce7 Added tag jdk8-b63 for changeset 3229597524ca ! .hgtags Changeset: dd1a80efa7cf Author:anthony Date: 2012-10-30 15:04 +0400 URL:

hg: jdk8/build/hotspot: 16 new changesets

2012-11-07 Thread david . katleman
Changeset: 4d37eb50b9b1 Author:katleman Date: 2012-11-01 14:11 -0700 URL: http://hg.openjdk.java.net/jdk8/build/hotspot/rev/4d37eb50b9b1 Added tag jdk8-b63 for changeset acabb5c282f5 ! .hgtags Changeset: a516debe2cee Author:amurillo Date: 2012-10-26 14:18 -0700 URL:

hg: jdk8/build/jaxp: Added tag jdk8-b63 for changeset 192d8a244bc3

2012-11-07 Thread david . katleman
Changeset: 27ab79568c34 Author:katleman Date: 2012-11-01 14:11 -0700 URL: http://hg.openjdk.java.net/jdk8/build/jaxp/rev/27ab79568c34 Added tag jdk8-b63 for changeset 192d8a244bc3 ! .hgtags

hg: jdk8/build/jaxws: Added tag jdk8-b63 for changeset 86989f702267

2012-11-07 Thread david . katleman
Changeset: 5ded18a14bcc Author:katleman Date: 2012-11-01 14:11 -0700 URL: http://hg.openjdk.java.net/jdk8/build/jaxws/rev/5ded18a14bcc Added tag jdk8-b63 for changeset 86989f702267 ! .hgtags

hg: jdk8/build/langtools: 17 new changesets

2012-11-07 Thread david . katleman
Changeset: 26831b6fcc4a Author:katleman Date: 2012-11-01 14:13 -0700 URL: http://hg.openjdk.java.net/jdk8/build/langtools/rev/26831b6fcc4a Added tag jdk8-b63 for changeset 92e6f2190ca0 ! .hgtags Changeset: 78962d89f283 Author:jjg Date: 2012-10-23 13:20 -0700 URL:

Re: Review Request: 8002028: build-infra: need no-hotspot partial build

2012-11-07 Thread David Holmes
Hi Erik, Two queries: 1. In Main.gmk wouldn't it be better to move the hotspot targets inside the conditional - that way if you ask to build hotspot when you are not configured to build it you will get an error. 2. Why do you have to import hotspot from a full JDK ?? David On 8/11/2012

hg: jdk8/build/jdk: 8002040: Allow Full Debug Symbols when cross-compiling

2012-11-07 Thread david . holmes
Changeset: 3717bcf9d7a7 Author:dholmes Date: 2012-11-07 23:12 -0500 URL: http://hg.openjdk.java.net/jdk8/build/jdk/rev/3717bcf9d7a7 8002040: Allow Full Debug Symbols when cross-compiling Reviewed-by: dcubed, erikj, tbell ! make/common/Defs-linux.gmk

Re: Review request: 8002365: build-infra: Build-infra fails on solaris 11.1 on sparc

2012-11-07 Thread David Holmes
On 8/11/2012 12:38 AM, Erik Joelsson wrote: This is essentially the patch from Vincent to make build-infra work on Solaris 11.1 with some minor adjustments to keep the new and old build equal. http://cr.openjdk.java.net/~erikj/8002365/webrev.01/

Re: Review request: 8002365: build-infra: Build-infra fails on solaris 11.1 on sparc

2012-11-07 Thread David Holmes
On 8/11/2012 4:56 PM, David Holmes wrote: On 8/11/2012 12:38 AM, Erik Joelsson wrote: This is essentially the patch from Vincent to make build-infra work on Solaris 11.1 with some minor adjustments to keep the new and old build equal. http://cr.openjdk.java.net/~erikj/8002365/webrev.01/