Looks OK to me.
SAM
On 05.11.2013 20:28, Sergey Bylokhov wrote:
Hello,
Updated version:
http://cr.openjdk.java.net/~serb/8027696/webrev.01/
Dates and spaces were fixed.
On 02.11.2013 15:37, Alan Bateman wrote:
On 01/11/2013 11:18, Sergey Bylokhov wrote:
Hello.
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:
http://cr.openjdk.java.net/~serb/8027696/webrev.00
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,
test/com/sun/jmx/remote/NotificationMarshalVersions/TestSerializationMismatch.java
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 range?
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.
-Alan.
