RFR (S): 8147943 jvmti.h generated with GPL header

[serguei.spitsyn at oracle.com]

Hi Alan,

It is a nice suggestion to use the <top>/make/templates/gpl-cp-header.
But it does not seem easy to reach it from the XSL processing.
So, I left the fix as it is.

Thanks,
Serguei


On 9/20/16 19:43, Alan Bateman wrote:
> On 20/09/2016 11:33, serguei.spitsyn at oracle.com wrote:
>> 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 have the 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.
> The patch looks okay to me but I just wonder if it would be worth 
> trying to consume license template from <top>/make/templates rather 
> than having it in jvmtiLib.xsl - you may have looked at this ready and 
> discounted it.
>
> -Alan
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/serviceability-dev/attachments/20160921/78c694ef/attachment.html>