[8u60] Request for backport approval: 8069057 and 8072732

Erik Joelsson erik.joelsson at oracle.com
Wed Mar 11 08:48:09 UTC 2015


The new patches look fine to me.

/Erik

On 2015-03-10 21:28, Dean Long wrote:
> I would like to backport "8069057: Make sure configure is run by bash" 
> and the followup fix "8072732: Regression in configure due to 
> JDK-8069057" to 8u60.  Both are needed by the previously approved 
> backport of "8064357: AARCH64: Top-level JDK changes". Without it, 
> JPRT fails when trying to use /bin/sh instead of /bin/bash.
>
> Bugs:
>
> https://bugs.openjdk.java.net/browse/JDK-8069057
> https://bugs.openjdk.java.net/browse/JDK-8072732
>
> Original review threads:
>
> http://mail.openjdk.java.net/pipermail/build-dev/2015-January/014064.html
> http://mail.openjdk.java.net/pipermail/build-dev/2015-February/014267.html 
>
>
> The jdk9 changesets are here:
>
> http://hg.openjdk.java.net/jdk9/dev/rev/da879ffb91cc
> http://hg.openjdk.java.net/jdk9/dev/rev/507654772146
>
> Unfortunately, the patches do not import cleanly.  Here are the
> modified patches:
>
> diff -r d9ba714b6e7f -r 43ed2e8bd6ea common/autoconf/configure
> --- a/common/autoconf/configure Mon Mar 09 14:20:35 2015 -0400
> +++ b/common/autoconf/configure Mon Mar 09 20:29:10 2015 -0400
> @@ -28,6 +28,9 @@
>    exit 1
>  fi
>
> +# Force autoconf to use bash.
> +export CONFIG_SHELL=$BASH
> +
>  CONFIGURE_COMMAND_LINE="$@"
>  conf_script_dir=`dirname $0`
>
> diff -r 43ed2e8bd6ea -r b5121ccd1458 common/autoconf/configure
> --- a/common/autoconf/configure Mon Mar 09 20:29:10 2015 -0400
> +++ b/common/autoconf/configure Mon Mar 09 20:30:23 2015 -0400
> @@ -28,8 +28,9 @@
>    exit 1
>  fi
>
> -# Force autoconf to use bash.
> +# Force autoconf to use bash. This also means we must disable 
> autoconf re-exec.
>  export CONFIG_SHELL=$BASH
> +export _as_can_reexec=no
>
>  CONFIGURE_COMMAND_LINE="$@"
>  conf_script_dir=`dirname $0`
>
> Let me know if there are any concerns about
> these backports.
>
> thanks,
>
> dl



More information about the jdk8u-dev mailing list