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

[Dean Long]

Thanks Erik.

dl

On 3/11/2015 1:48 AM, Erik Joelsson wrote:
> 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
>