Create a jvm.cfg for zero on 32 bit architectures

[David Holmes]

Hi Omair,

On 10/08/2013 1:25 AM, Omair Majid wrote:
> Hi,
>
> OpenJDK fails to build with zero currently on 32 bit architectures.
> There's a non-architecture-specific compilation error that I fixed [1].
> That makes zero build/run on 64-bit architectures. However, the build
> still fails on 32-bit. The error says that a jvm.cfg can not be found
> for zero.
>
> The following webrev fixes it:
> http://cr.openjdk.java.net/~omajid/webrevs/zero-jvm-config/00/
>
> It creates a jvm.cfg that's identical to the one used for 64-bit [2].

I'm not sure this is the "right" way to fix this as I would think zero 
should always just use the jvm.cfg that is in the repository. But the 
existing logic makes that awkward. I think what we should have is 
something like (I can't recall the right way to express this off the top 
of my head):

CLIENT_AND_SERVER := ...

COPY_JVM_CFG_FILE := CLIENT_AND_SERVER == true || BITS == 64 ||
                      JVM_VARIANT_ZERO == true

ifeq ($(COPY_JVM_CFG_FILE),true)
      $(JVMCFG): $(JVMCFG_SRC)
          $(call install-file)
else
      $(JVMCFG):
          $(MKDIR) -p $(@D)
          $(RM) $(@)

      # Now check for other permutations
...

Your change is less intrusive in the sense that it can't affect the 
other JVM variants.

That aside do you not want to also address zeroshark? Or is that only 
64-bit?

David

> Thanks,
> Omair
>
> [1] http://hg.openjdk.java.net/hsx/hotspot-rt/hotspot/rev/c54a3122f9c8
> [2]
> http://hg.openjdk.java.net/jdk8/build/jdk/file/8ed8e2b4b90e/src/solaris/bin/zero/jvm.cfg
>