Review Request: Build-infra M1

[Andrew Hughes]

----- Original Message -----
> As outlined in [1], the build-infra project would like to push the
> current work into jdk8 in order to expose it to a wider audience.
> The webrevs are made against the jdk8/build forest. In each
> repository, there are two kinds of changes:
> 
> 1. Changes to old makefiles and source code to be compatible with the
> new build.
> 2. The new makefiles
> 
> For corba, jaxp and jaxws, all changes of category 1 have already
> gone in. For langtools, we are awaiting one more change for
> introducing the GenerateNativeHeader annotation. For hotspot, all
> necessary changes have been pushed into hotspot-rt. For jdk, there
> are two webrevs, one with everything and one with just the category
> 1 changes, to make it easier to see them. Finally for the root
> repository there are only new files in the common subdir.
> 
> root, configure script and makefiles:
> http://cr.openjdk.java.net/~erikj/build-infra-m1/webrev-root-new/
> 
> langtools, 1 new makefile:
> http://cr.openjdk.java.net/~erikj/build-infra-m1/webrev-langtools-new/
> 
> langtools, GenerateNativeHeader annotation (this is already going in
> through tools, but adding it here for reference as the jdk changes
> depend on it)
> http://cr.openjdk.java.net/~erikj/build-infra-m1/webrev-langtools-nativeheader/
> 
> corba, 1 new makefile:
> http://cr.openjdk.java.net/~erikj/build-infra-m1/webrev-corba-new/
> 
> jaxp, 1 new makefile
> http://cr.openjdk.java.net/~erikj/build-infra-m1/webrev-jaxp-new/
> 
> jaxws, 1 new makefile
> http://cr.openjdk.java.net/~erikj/build-infra-m1/webrev-jaxws-new/
> 
> jdk, just the changes to old files
> http://cr.openjdk.java.net/~erikj/build-infra-m1/webrev-jdk-other/
> 
> jdk, all changes including a partial copy of the old makefiles.
> http://cr.openjdk.java.net/~erikj/build-infra-m1/webrev-jdk-new/
> 
> Of course, if you prefer you can look at the new makefiles directly
> in the build-infra/jdk8 repository forest too.
> 
> These changes should not affect the old build at all. To build using
> the new build system, change directory to "common/makefiles" and:
> 
> ../autoconf/configure
> make
> (make images)
> 
> State of the new build (the old build should of course be
> unaffected):
> 
> 
>     • Linux 32bit: Works
>     • Linux 64bit: Works
>     • Windows 32bit: Works
>     • Windows 64bit: Works
>     • Solaris i586: Builds but launchers currently unusable
> 
> Some notes:
> 
> 
>     • The old and new build (on linux x64) produce very close to
>     equal results. There is a comparison script in
>     common/bin/compareimage.sh with which this can be checked.
>     • Not all makefiles in jdk have been converted yet, for those
>     that haven't been, a copy of the old files are used.
>     • Not all promised features in the java compilation are active
>     and ready in this milestone. Most notably, it's still not using
>     more than one cpu and the nifty new dependency tracking is
>     disabled. A clean build is still pretty fast, but incremental
>     builds aren't as good as they will be yet.
>     • On windows, only cygwin is currently supported.
> 
> Now please share your feedback!
> 
> /Erik
> 
> [1]
> http://mail.openjdk.java.net/pipermail/build-infra-dev/2012-March/000571.html
> 
> 

I haven't tried this out yet, but I'll try and give it a spin in build-infra.

One thing that did stand out from the patch is that generated files such as
configure are being checked in.  For updates to this, is there a plan to
mandate the use of a specific version of autoconf?  Otherwise, we're going
to get changes simply because someone generated the file using a different
version.  Alternatively, we could just require that the user has autoconf
installed and runs it prior to configure.
-- 
Andrew :)

Free Java Software Engineer
Red Hat, Inc. (http://www.redhat.com)

PGP Key: 248BDC07 (https://keys.indymedia.org/)
Fingerprint = EC5A 1F5E C0AD 1D15 8F1F  8F91 3B96 A578 248B DC07