RFR (S): 8006965: test_gamma should run with import JDK

David Holmes david.holmes at oracle.com
Fri Feb 22 04:10:23 UTC 2013


Hi Christian,

cc'ing build-dev

On 22/02/2013 12:40 PM, Christian Thalinger wrote:
> http://cr.openjdk.java.net/~twisti/8006965
>
> 8006965: test_gamma should run with import JDK
> Reviewed-by:
>
> Right now test_gamma runs with the boot JDK which is JDK n-1 (where
> JDK n is the version we are actually compiling for).  This setup is
> unsupported and thus should not be done during HotSpot builds.
>
> The fix is to always use JDK_IMPORT_PATH instead of JAVA_HOME when
> running test_gamma.

First, the simplest change seems to me to just to do:

! MAKE_ARGS += JAVA_HOME=$(JDK_IMPORT_PATH)

and not need to change anything in buildtree.make.

But this forces everyone outside of Oracle (and some inside) to have to 
set ALT_JDK_IMPORT_PATH explicitly otherwise they get:

JDK_IMPORT_PATH=$(SLASH_JAVA)/re/j2se/$(JDK_VERSION)/promoted/latest/binaries/$(PLATFORM)

And the boot JDK is only JDK n-1 for our official builds - people can 
use whatever boot JDK works for them including latest JDK.

I see the problem you want to fix, and I know why you've hit it, but 
this doesn't seem like a general solution. I don't know what to suggest.

> make/bsd/makefiles/buildtree.make
> make/defs.make

Why did you add:

MAKE_ARGS += BOOTDIR=$(ABS_BOOTDIR)

?

David
-----

> make/linux/makefiles/buildtree.make
> make/solaris/makefiles/buildtree.make
>



More information about the build-dev mailing list