<AWT Dev>  Review Request: 8027696 Incorrect copyright header in the tests
Sergey.Bylokhov at oracle.com
Tue Nov 5 08:28:03 PST 2013
Dates and spaces were fixed.
On 02.11.2013 15:37, Alan Bateman wrote:
> On 01/11/2013 11:18, Sergey Bylokhov wrote:
>> Please review the fix for jdk 8.
>> Most of tests in the sound area, and some tests in the client,
>> java.lang, security, jmx etc has incorrect copyright.
>> According to the http://openjdk.java.net/faq
>> "GPL v2 + the Classpath exception for the class libraries and those
>> parts of the virtual machine that expose public APIs"
>> But currently our tests mix gpl+cp and gpl or has no header at all.
>> Tests with "/nodynamiccopyright/", with other company's copyright, or
>> other gpl templates were not updated.
>> Also it would be good if in the faq we explicitly mention about
>> copyright of the tests.
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8027696
>> Webrev can be found at:
> Thanks for doing this. I sampled a few of the files in the webrev and
> the changes mostly look okay.
> The only thing is that I'm not sure about is the dates that you've put
> on tests that were missing a header. For example,
> was added in 2013 but the header proposed in the webrev has 2012. Does
> your script to fix up these headers look at the hg log to get the date
> I see you've changed jdk/test/Makefile to have the GPL header. I don't
> know what the right header is for that (as the rest of the Makefiles
> have the GPL + Classpath exception).
> One other question is whether this is a one-off effort by yourself or
> whether this is part of an effort to keep us in check on a continuous
> basis. Periodically David Katleman brings up malformed headers on
> jdk8-dev. I assume these are caught by something that checks the
> headers on a weekly or continuous basis. Maybe there is an opportunity
> to combine efforts and also have these scripts run on a continuous
> basis on jdk8/tl, jdk8/awt and the other forests that collect changes.
Best regards, Sergey.
More information about the awt-dev