[8u] RFR: 8038340: Cleanup and fix sysroot and devkit handling on Linux and Solaris

[Tim Bell]

In that case, the review looks good.

Thanks-

Tim

On 04/13/18 13:15, Kevin Walls wrote:
> Hi Erik - thanks for clarifying.
>
>
> On 13/04/2018 20:58, Erik Joelsson wrote:
>> ccache shouldn't be commented out. That must have been a local edit
>> mistake in the original webrev.
>>
>> /Erik
>>
>>
>> On 2018-04-13 12:51, Kevin Walls wrote:
>>> Thanks Tim -
>>>
>>> There's a later webrev in the review thread:
>>> http://cr.openjdk.java.net/~erikj/8038340/webrev.root.03/ in which I
>>> see the common/autoconf/configure.ac change... It's in the commit in
>>> 9 also.  I think we need it. 8-)
>>>
>>> But that webrev.03 also has the commenting out of "Building
>>> ccache..." in make/devkit/Makefile , though I don't see it happening
>>> in 8038340 in the 9 change, or in latest jdk either. I'm thinking
>>> that ccache was disabled for local testing and wasn't intended as
>>> part of 8038340 - do let me know if you think otherwise!
>>>
>>> Thanks
>>> Kevin
>>>
>>>
>>> On 13/04/2018 20:08, Tim Bell wrote:
>>>> Kevin - looks good in general with a few remarks (see below):
>>>>
>>>>> I'd like to request review of this backport from 9 to 8u:
>>>>>
>>>>> 8038340: Cleanup and fix sysroot and devkit handling on Linux and
>>>>> Solaris
>>>>> JBS: https://bugs.openjdk.java.net/browse/JDK-8038340
>>>>>
>>>>> 9 changesets:
>>>>> base repo: http://hg.openjdk.java.net/jdk9/dev/rev/9786ef8ca58c
>>>>> JDK: repo: http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/fdeb6a8b0f3a
>>>>> (clean import to jdk8u-dev)
>>>>>
>>>>> JDK repo change imports cleanly.  base repo changes require some
>>>>> manual
>>>>> fixups in:
>>>>>
>>>>> common/autoconf/basics.m4 addition of
>>>>> AC_DEFUN_ONCE([BASIC_SETUP_DEVKIT], fails, done manually
>>>>> common/autoconf/toolchain.m4 minor change
>>>>> didn't import
>>>>> make/common/NativeCompilation.gmk                minor change
>>>>> didn't import
>>>>>
>>>>> ...plus run autogen to recreate generated files.
>>>>>
>>>>>
>>>>> 9 review thread:
>>>>> http://mail.openjdk.java.net/pipermail/2d-dev/2014-April/004361.html
>>>>> (thread starts in March, not that it's long, but I don't see a link
>>>>> there to the previous emails in the thread:
>>>>> http://mail.openjdk.java.net/pipermail/2d-dev/2014-March/004333.html )
>>>>>
>>>>> 8u webrev:http://cr.openjdk.java.net/~kevinw/8038340/webrev.00/
>>>>>
>>>>> Many thanks!
>>>>> Kevin
>>>>
>>>> common/autoconf/configure.ac
>>>>
>>>> This change looks straightforward, but I don't see the file at all
>>>> in the JDK 9 webrev...
>>>>
>>>>
>>>> make/common/autoconf/configure.acdevkit/Makefile
>>>>
>>>> lines 91,92 ... Do you want ccache?  It is commented out in the JDK
>>>> 9 changes
>>>> (http://cr.openjdk.java.net/~erikj/8038340/webrev.root.02/make/devkit/Makefile.frames.html)