[8u-dev] Request for approval: 8031759: Configure should handle overrides of tools better

[Kevin Walls]

Great,thanks!


On 21/03/2018 01:04, Rob McKenna wrote:
> Approved
>
>      -Rob
>
> On 20/03/18 16:38, Kevin Walls wrote:
>> Hi,
>>
>> I'd like to request approval to backport to jdk8u:
>>
>> 8031759: Configure should handle overrides of tools better
>> JBS: https://bugs.openjdk.java.net/browse/JDK-8031759
>> 9 changeset: http://hg.openjdk.java.net/jdk9/jdk9/rev/82b65f34b79d
>>
>> 8u webrev:
>> http://cr.openjdk.java.net/~kevinw/8031759/webrev.00/
>>
>> This is mostly straightforward, large parts import cleanly, the parts that
>> didn't are changes to macro names,  and a couple of tools referenced in the
>> 9 change which are not in 8u (OTOOL and LIPO).
>>
>> Also need to take into account this change (avoid creating this problem) :
>> 8033115: Bad merge between 8027584 and 8031759
>> https://bugs.openjdk.java.net/browse/JDK-8033115
>>
>> Which is just this edit in common/autoconf/build-performance.m4, included
>> here in the proposed 8u push.
>> -    BASIC_REQUIRE_PROG(CCACHE, ccache)
>> +    BASIC_REQUIRE_PROGS(CCACHE, ccache)
>>
>> (I can mark that bug as not present in 8 when done)
>>
>> Then with the .m4 changes in place, ran common/autoconf/autogen.sh to
>> recreate common/autoconf/generated-configure.sh which is included in the
>> webrev/change.
>>
>> Thanks!
>> Kevin
>>