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

[Kevin Walls]

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)
>>>
>>>
>>>
>>> /Tim
>>>
>>
>