Fwd: hg: hsx/hotspot-rt/hotspot: 7141242: build-infra merge: Rename CPP->CXX and LINK->LD

2012-02-06 Thread David Holmes
Sorry for the wide distribution but the handful of people that may be affected by this could be spread across various groups. The patch below changes the names of the some of make variables used when building hotspot. Eg CPPFLAGS becomes CXXFLAGS, LINKXXX becomes LDXXX etc. It is possible th

Re: Request for review: hotspot/jprt.properties [S]

2012-02-06 Thread David Holmes
On 3/02/2012 9:55 PM, Keith McGuigan wrote: On 2/2/2012 9:01 PM, David Holmes wrote: On 3/02/2012 9:56 AM, Keith McGuigan wrote: Here's a webrev for changing the default JPRT release value for hotspot to jdk8. It can still be overridden by the -release jdk7 during JPRT submission. I also remove

Re: Review Request : 7143162 : Allow disable building of jdk demos and samples

2012-02-06 Thread Kelly O'Hair
Looks ok to me. -kto On Feb 6, 2012, at 2:55 PM, Mike Duigou wrote: > Hello all; > > WEBREV @ http://cr.openjdk.java.net/~mduigou/7143162/0/webrev/ > > A fairly simple patch for review. It allows developers to shut off building > of the jdk demos and samples. Disabling these builds has a ver

Review Request : 7143162 : Allow disable building of jdk demos and samples

2012-02-06 Thread Mike Duigou
Hello all; WEBREV @ http://cr.openjdk.java.net/~mduigou/7143162/0/webrev/ A fairly simple patch for review. It allows developers to shut off building of the jdk demos and samples. Disabling these builds has a very positive impact upon build turnaround for incremental builds. Export NO_DEMOS and

Re: Adding asm to JDK 8

2012-02-06 Thread Kelly O'Hair
On Feb 6, 2012, at 8:46 AM, Jim Holmlund wrote: > > > On 2/6/2012 12:04 AM, David Holmes wrote: >> On 4/02/2012 5:03 AM, Jim Holmlund wrote: >>> On 2/2/2012 7:13 PM, David Holmes wrote: I added build-dev to the cc as this is of general interest. Actually before doing this a heads-up s

Re: Adding asm to JDK 8

2012-02-06 Thread Jim Holmlund
On 2/6/2012 12:04 AM, David Holmes wrote: On 4/02/2012 5:03 AM, Jim Holmlund wrote: On 2/2/2012 7:13 PM, David Holmes wrote: I added build-dev to the cc as this is of general interest. Actually before doing this a heads-up should go out to all jdk8-dev I think. One question: are there any cr

Re: Review for 7141242: build-infra merge: Rename CPP->CXX and LINK->LD

2012-02-06 Thread Erik Joelsson
New webrev posted: http://cr.openjdk.java.net/~erikj/7141242/webrev.02/ Fredriks change for ccache added this line to all vm.make files: vm_version.o: CPPFLAGS += ${JRE_VERSION} which had to be replaced by vm_version.o: CXXFLAGS += ${J

Re: Adding asm to JDK 8

2012-02-06 Thread David Holmes
On 4/02/2012 5:03 AM, Jim Holmlund wrote: On 2/2/2012 7:13 PM, David Holmes wrote: I added build-dev to the cc as this is of general interest. Actually before doing this a heads-up should go out to all jdk8-dev I think. One question: are there any cross-compilation issues with building this? I'