hg: jdk8/build: 2 new changesets

2012-06-08 Thread kelly . ohair
Changeset: dd596160b323 Author:ohair Date: 2012-06-08 17:25 -0700 URL: http://hg.openjdk.java.net/jdk8/build/rev/dd596160b323 7170091: Fix missing wait between repo cloning in hgforest.sh Reviewed-by: strarup ! make/scripts/hgforest.sh Changeset: c61921f9b965 Author:ohair Date

Re: RFR: 7175331 - Remove FDS files from embedded JRE images

2012-06-08 Thread Kelly O'Hair
Looks fine. -kto On Jun 8, 2012, at 3:19 PM, David Holmes wrote: > cc'ing build-dev for a review by a 7u Reviewer. > > Thanks, > David > > On 8/06/2012 11:01 AM, David Holmes wrote: >> This is a simple fix that only applies to 7u and is targeted for 7u6. >> >> We need to update the embedded i

Re: RFR: 7175331 - Remove FDS files from embedded JRE images

2012-06-08 Thread David Holmes
cc'ing build-dev for a review by a 7u Reviewer. Thanks, David On 8/06/2012 11:01 AM, David Holmes wrote: This is a simple fix that only applies to 7u and is targeted for 7u6. We need to update the embedded image target in Release-embedded.gmk to remove the FDS related files that now get create

Re: Missing/wrong build dependencies for inline functions in HotSpot

2012-06-08 Thread Volker Simonis
Yes, that's really strange. You're right, the dependency file should contain ".. the names of all the included files" (gcc -man page). So it seems to be a bug in gcc and how it handles '-MMD' although I couldn't find a bug report for it and I can't believe that nobody else has noticed this before.

Re: Need reviewer: build-infra makefile changes

2012-06-08 Thread Kelly O'Hair
Technically, I had plenty of reviewers via the build-infra team, my wording was a bit off on this email. Sorry. These new build-infra changes have now been pushed into the jdk8/build forest. I have NOT verified the build-infra builds, only verified that the existing old makefiles are not impact

hg: jdk8/build/jaxws: 7170079: Adjustments to build-infra makefiles

2012-06-08 Thread kelly . ohair
Changeset: 001351a98bd5 Author:erikj Date: 2012-06-07 20:30 -0700 URL: http://hg.openjdk.java.net/jdk8/build/jaxws/rev/001351a98bd5 7170079: Adjustments to build-infra makefiles Reviewed-by: ohair, ohrstrom, ihse, jonas Contributed-by: jonas , erikj , ihse , tgranat , ykantser !

hg: jdk8/build/jdk: 3 new changesets

2012-06-08 Thread kelly . ohair
Changeset: c499fd3f1695 Author:erikj Date: 2012-06-07 18:05 -0700 URL: http://hg.openjdk.java.net/jdk8/build/jdk/rev/c499fd3f1695 7170969: Add @GenerateNativeHeader to classes whose fields need to be exported for JNI Reviewed-by: ohair, ohrstrom, ihse ! make/sun/awt/make.depend !

hg: jdk8/build/langtools: 7170079: Adjustments to build-infra makefiles

2012-06-08 Thread kelly . ohair
Changeset: ac29cc95b5c5 Author:erikj Date: 2012-06-07 20:30 -0700 URL: http://hg.openjdk.java.net/jdk8/build/langtools/rev/ac29cc95b5c5 7170079: Adjustments to build-infra makefiles Reviewed-by: ohair, ohrstrom, ihse, jonas Contributed-by: jonas , erikj , ihse , tgranat , ykantser

hg: jdk8/build/corba: 7170079: Adjustments to build-infra makefiles

2012-06-08 Thread kelly . ohair
Changeset: 0c0b50c7b76a Author:erikj Date: 2012-06-07 20:30 -0700 URL: http://hg.openjdk.java.net/jdk8/build/corba/rev/0c0b50c7b76a 7170079: Adjustments to build-infra makefiles Reviewed-by: ohair, ohrstrom, ihse, jonas Contributed-by: jonas , erikj , ihse , tgranat , ykantser !

hg: jdk8/build/jaxp: 7170079: Adjustments to build-infra makefiles

2012-06-08 Thread kelly . ohair
Changeset: 83a38059327b Author:erikj Date: 2012-06-07 20:30 -0700 URL: http://hg.openjdk.java.net/jdk8/build/jaxp/rev/83a38059327b 7170079: Adjustments to build-infra makefiles Reviewed-by: ohair, ohrstrom, ihse, jonas Contributed-by: jonas , erikj , ihse , tgranat , ykantser !

hg: jdk8/build: 7170079: Adjustments to build-infra makefiles

2012-06-08 Thread kelly . ohair
Changeset: efd26e051e50 Author:erikj Date: 2012-06-07 20:25 -0700 URL: http://hg.openjdk.java.net/jdk8/build/rev/efd26e051e50 7170079: Adjustments to build-infra makefiles Reviewed-by: ohair, ohrstrom, ihse, jonas Contributed-by: jonas , erikj , ihse , tgranat , ykantser ! READM

Re: Missing/wrong build dependencies for inline functions in HotSpot

2012-06-08 Thread Keith McGuigan
I don't understand why gcc doesn't put frame_x86.inline.hpp into the generated/dependencies/frame.o.d file. Isn't the point of -MMD to calculate the full closer of header files used for listing as a dependency? Is this a bug in gcc or are we using it wrong? I notice that Sun Studio compile

Missing/wrong build dependencies for inline functions in HotSpot

2012-06-08 Thread Volker Simonis
Hi, I've just stumbled across the problem that changing the implementation of an inline function in HotSpot does not necessarily rebuild all the call sites of that function. This is because because of the way how the build dependencies are handled within the HotSpot. As an example you may have a l