FYI: Fix PR632: patches/security/20110215/6878713.patch breaks shark zero build

Mark Wielaard mark at klomp.org
Tue Feb 22 01:39:09 PST 2011


On Mon, 2011-02-21 at 20:52 +0000, Dr Andrew John Hughes wrote:
> I can see how it would be advantageous from a speed perspective, but for me,
> if I'm going to bother building one of the alternative VMs, I want to know it
> actually works.  Now I guess you can run 'check-hotspot' with each of the alternative
> VMs but I don't see why that saves anything over just doing the full build,
> which tests the VM heavily with lots of javac invocations.

So we need targets for at least check-hotspot and check-langtools to be
run with alternative runtimes build in. This might even be a nice idea
for pure hotspot client/server/tiered testing. Ideally also
make-check-jdk, but that on itself takes ages.

> > Also, could we have an alias for "hotspot=next" or something? Assuming
> > that we only support "current" and "next" hotspot, then you don't have
> > to remember which particular version it represents (also easier for
> > making sure a buildslave tracks them).
> > 
> 
> I think what you want is already catered for; as documented in README,
> a simple --with-hotspot-build will get you the alternative HotSpot.
> 
>   ORIGINAL_BUILD="original"
>   ALTERNATE_BUILD="hs20"
> ...
>    if test "x${HSBUILD}" = xyes; then
>         HSBUILD="${ALTERNATE_BUILD}"
>   elif test "x${HSBUILD}" = xno; then
>         HSBUILD="${ORIGINAL_BUILD}"
>   fi
> 
> so the two alternatives are always available via --with-hotspot-build
> and --without-hotspot-build, the former being the one from the HotSpot
> tarball, the latter from the original OpenJDK tarball.

O wow. That is nice. Completely missed that. I added a new "quick"
squeeze-x86_64 builder icedtea6-squeeze-x86_64-quick-hotspot for that
(patch attached - note the horrible copy/paste, sorry about that).

I would almost suggest you make this so that it is possible to use this
with --enable-additional-vms for easy testing by users...

Cheers,

Mark
-------------- next part --------------
A non-text attachment was scrubbed...
Name: icedtea6-squeeze-x86_64-quick-hotspot.patch
Type: text/x-patch
Size: 3076 bytes
Desc: 
Url : http://mail.openjdk.java.net/pipermail/distro-pkg-dev/attachments/20110222/391e1b35/icedtea6-squeeze-x86_64-quick-hotspot.patch 


More information about the distro-pkg-dev mailing list