are there changes for building mlvm now that 1.7 is released?

[John Rose]

On Sep 11, 2011, at 1:10 AM, Michael Barker wrote:

>> Please give it a whirl and let me know what breaks.
> 
> I have a working build on Mac OS X Lion with gcc-4.2.  I started with
> stephenb's MLVM build scripts.

Excellent work.

Did you have to use patches/hotspot/snowleopard.patch for gcc-4.2, or can we delete that?

> I have a couple of small issues to
> crack as I went:
> 
> - Bootstrapped with 64bit bsd-port openjdk 1.7 binaries from:
> http://code.google.com/p/openjdk-osx-build/.

Yes.  Now that the sources are for JDK 8, they require a JDK 7 installation to build from.

For Mac, this could require doing a separate build for the bootstrap JDK, or altering the sanity check logic.  I used a saved JDK 7 build.

> - Disabled warning as errors as per:
> http://wikis.sun.com/display/OpenJDK/Darwin10Build

Good.

> - For some reason the /usr/bin/printf binary didn't like one of the
> printf statements in the top-level make folder, so the made the
> following change:
> 
> --- a/make/Defs-internal.gmk	Thu Sep 01 13:54:18 2011 -0700
> +++ b/make/Defs-internal.gmk	Sun Sep 11 08:59:09 2011 +0100
> @@ -79,7 +79,7 @@
> # Find all build_time_* files and print their contents in a list sorted
> # on the name of the sub repository.
> define ReportBuildTimes
> -$(PRINTF) "-- Build times ----------\nTarget %s\nStart %s\nEnd
> %s\n%s\n%s\n-------------------------\n" \
> +$(PRINTF) "## Build times ##########\nTarget %s\nStart %s\nEnd
> %s\n%s\n%s\n#########################\n" \
> 
> For some reason printf was interpreting the initial "--" as command
> line argument and failed stating that "-" was not a valid option.

Oops, this is a new change in bsd-port.  I just moved it to the repo mlvm.

> - The really odd one was the hotspot build wasn't copying the
> necessary files into its import folder (failed to find the appropriate
> build target for the copy).  I tracked it down the the
> hotspot/make/Makefile export folder settings weren't picking up the
> values for HS_ARCH and VM_PLATFORM from the
> hotspot/make/bsd/makefiles/defs.make file.  I hacked (i.e. I wouldn't
> consider it a proper fix) HS_ARCH=x86 and VM_PLATFORM=bsd_amd64 into
> the top of the file and it all built correctly.

That smells like a failed attempt to run "uname" somewhere.
What did this line do in defs.make:
   ARCH:=$(shell uname -m)

-- John