RFR (S): 8147943 jvmti.h generated with GPL header
serguei.spitsyn at oracle.com
serguei.spitsyn at oracle.com
Tue Sep 20 18:33:09 UTC 2016
Please, review the fix for:
https://bugs.openjdk.java.net/browse/JDK-8147943
Webrev:
http://cr.openjdk.java.net/~sspitsyn/webrevs/2016/hotspot/8147943-jvmti-header.hs1/
Summary:
The problem is that the
build/*/hotspot/variant-server/gensrc/jvmvtifiles/jvmti.h
is currently generated with the GPL copyright comment.
The JDK version of the header that we ship must havethe GPL +
"Classpath" exception.
So that the file is taken from the version that is checked into the
jdk repository:
jdk/src/java.base/share/native/include/jvmti.h.
Now, the checked-in version of the jvmti.h in the jdk repository is
manually updated
from the jvmvtifiles/jvmti.h with the copyright comment replacement
(very inconvenient).
Replacement of the copyright comment in the jvmvtifiles/jvmti.h was
discussed
Alan and Iris, and they are Ok with the change.
The jvmti.h is generated from the
hotspot/src/share/vm/prims/jvmti.xml with the XSL scripts,
and its copyright comment is inherited from the jvmti.xml.
The fix is to update the XSL scripts to generate the GPL+CP copyright
comment.
The only part that is still taken from the jvmti.xml is the copyright
year line.
There is a separate bug that targets automatic installation of the
generated jvmti.h
and potential removal of the checked-in version from the jdk repository:
https://bugs.openjdk.java.net/browse/JDK-8063154
Checked in jvmti.h not in sync with generated jvmti.h
Testing:
Checked the copyright comment in the generated
gensrc/jvmvtifiles.jvmti.h.
Thanks,
Serguei
More information about the hotspot-dev
mailing list