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

9 changesets:
base repo:
JDK: repo:
(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 run autogen to recreate generated files.

9 review thread:
(thread starts in March, not that it's long, but I don't see a link
there to the previous emails in the thread: )

8u webrev:

Many thanks!


This change looks straightforward, but I don't see the file at all in the JDK 9 webrev...


lines 91,92 ... Do you want ccache? It is commented out in the JDK 9 changes (


Reply via email to