kulla fails to configure with Java 9 ea (Early Access)

[ShinyaYoshida]

Hi Samuel, thank you for resending.

Oops, let me confirm one thing please.
You got this error on a repository hosted on kulla-dev, right?

If so, unfortunately it isn't under maintaining because it already merged
to jdk9-dev.
Please confirm that you still get this error on jdk9-dev and your patch
works well on it.

Regards,
shinyafox(Shinya Yoshida)

2016/07/29 午後4:55 "Samuel Marks" <samuelmarks at gmail.com>:

> Error:
> configure: Found potential Boot JDK using configure arguments
> configure: Potential Boot JDK found at /opt/Java/jdk-9 is incorrect JDK
> version (java version "9-ea"); ignoring
> configure: (Your Boot JDK must be version 8 or 9)
> configure: error: The path given by --with-boot-jdk does not contain a
> valid Boot JDK
>
> Here's a fix to the kulla repository (also attached, not sure what the
> protocol is):
>
> # HG changeset patch
> # User Samuel Marks <samuelmarks at gmail.com>
> # Date 1469777474 -36000
> #      Fri Jul 29 17:31:14 2016 +1000
> # Node ID 9268348574206a8487901c8e8a1ea15987709521
> # Parent  77a1e3986f77f6e9d1037f4de8296d927ef12ac4
> New version check routine to handle Java 9 Early Access release
>
> diff -r 77a1e3986f77 -r 926834857420 common/autoconf/boot-jdk.m4
> --- a/common/autoconf/boot-jdk.m4    Sat Aug 22 00:47:29 2015 -0700
> +++ b/common/autoconf/boot-jdk.m4    Fri Jul 29 17:31:14 2016 +1000
> @@ -74,11 +74,12 @@
>            BOOT_JDK_FOUND=no
>          else
>            # Oh, this is looking good! We probably have found a proper JDK.
> Is it the correct version?
> -          BOOT_JDK_VERSION=`"$BOOT_JDK/bin/java" -version 2>&1 | head -n
> 1`
> +          BOOT_JDK_VERSION=`"$BOOT_JDK/bin/java" -version 2>&1 | awk
> '/version/{print $NF}'`
> +          BOOT_JDK_VERSION="${BOOT_JDK_VERSION:1:3}"
> +          CORRECT_VERSIONS=(1.8 1.9 9 9-e)
>
>            # Extra M4 quote needed to protect [] in grep expression.
> -          [FOUND_CORRECT_VERSION=`echo $BOOT_JDK_VERSION | grep
> '\"1\.[89]\.'`]
> -          if test "x$FOUND_CORRECT_VERSION" = x; then
> +          if [[ ! " ${CORRECT_VERSIONS[*]} " == *" $BOOT_JDK_VERSION "*
> ]]; then
>              AC_MSG_NOTICE([Potential Boot JDK found at $BOOT_JDK is
> incorrect JDK version ($BOOT_JDK_VERSION); ignoring])
>              AC_MSG_NOTICE([(Your Boot JDK must be version 8 or 9)])
>              BOOT_JDK_FOUND=no
>
> Samuel Marks
> http://linkedin.com/in/samuelmarks
>
> BTW: I posted in kulla-dev but they told me to post here -
> http://mail.openjdk.java.net/pipermail/kulla-dev/2016-July/001508.html
>