Hi Erik,

On 14/03/2012 12:19 AM, Erik Joelsson wrote:
http://cr.openjdk.java.net/~erikj/7141246/webrev.02

New webrev posted. Removed all renames of KERNEL->MINIMAL. We would
still like to keep the name of the new JVM_VARIANT variable if possible.

I find it confusing to convert KERNEL_BUILD->(JVM_VARIANT_MINIMAL=true) and then check for JVM_VARIANT_MINIMAL, instead of just defining JVM_VARIANT_KERNEL. Particularly when KERNEL is used in a bunch of other variables.

make/Makefile

Are the Shark/Zero folk okay with the shark/zero changes? If I understand correctly, today if SHARK_BUILD==true then ZERO_BUILD==true, but in the new scheme JVM_VARIANT_ZEROSHARK and JVM_VARIANT_ZERO are distinct (but the name ZEROSHARK implies ZERO and SHARK).

---

/make/defs.make

  74   ifeq ($(ZERO_BUILD)$(KERNEL_BUILD)$(SHARK_BUILD),)
  75     # A default is needed
  76     ifeq ($(BUILD_CLIENT_ONLY), true)
  77       JVM_VARIANTS:=client
  78       JVM_VARIANT_CLIENT:=true
  79     endif
  80     # Further defaults are platform and arch specific

I don't understand why we would only need a default in the BUILD_CLIENT_ONLY case. Is this just to avoid replicating the BUILD_CLIENT_ONLY check in the platform specific makefiles?

David
-----


/Erik
You seem to have included the MINIMAL VM changes in here. Those changes
have not been pushed to the mainline and are not yet to be pushed. If
you want to push these build-infra changes you will need to factor out
the MINIMAL VM stuff - else wait.

David

Reply via email to