On 2017-12-04 12:17, David Holmes wrote:
Hi Magnus,
On 4/12/2017 9:06 PM, Magnus Ihse Bursie wrote:
JDK-8190484 was created as a follow-up bug to the unification of the
duplicated jvm.h, jvm_md.h and jmm.h, to determine the proper
location of these files. This has now been decided to be
hotspot/share/include and hotspot/os/$OS/include, respectively.
This patch moves the relevant files there, but since this also frees
up the src/$MODULE/native/include directories for the original
purpose, it also unifies and simplifies the build logic for these
directories, so that common code is executed for all modules to just
copy any exported header files from these directories, should they
exist.
I'm intending to push this to jdk-hs.
Do you want this in 10 or 11? If you want 10 then you need to push to
jdk/jdk as there may not be another snapshot from jdk/hs.
It was targeted at 11 in the JBS issue. I don't have a strong opinion on
forcing this into JDK 10. Also, one of the fixes it depended on was not
yet integrated from jdk/hs to jdk/jdk.
But if you think it should go into JDK 10 and jdk/jdk, just let me know
and I'll move the changeset there, and await the next jdk/hs -> jdk/jdk
integration.
/Magnus
Bug: https://bugs.openjdk.java.net/browse/JDK-8190484
WebRev:
http://cr.openjdk.java.net/~ihse/JDK-8190484-move-hotspot-exported-includes/webrev.01
Seems okay for the jvm.h/jmm.h changes. I couldn't follow all the flow
on cleanups.
Thanks,
David
/Magnus
