[8] Review Request: 8027696 Incorrect copyright header in the tests

2013-11-01 Thread Sergey Bylokhov
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 mach

Re: [8] Review Request: 8027696 Incorrect copyright header in the tests

2013-11-01 Thread Sean Mullan
The security tests look fine. --Sean On 11/01/2013 07:18 AM, 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 +

Re: [8] Review Request: 8027696 Incorrect copyright header in the tests

2013-11-02 Thread Alan Bateman
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 l

Re: [8] Review Request: 8027696 Incorrect copyright header in the tests

2013-11-02 Thread Sergey Bylokhov
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

Re: [8] Review Request: 8027696 Incorrect copyright header in the tests

2013-11-05 Thread Sergey Bylokhov
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, j

Re: [8] Review Request: 8027696 Incorrect copyright header in the tests

2013-11-07 Thread Alan Bateman
On 05/11/2013 16:28, Sergey Bylokhov wrote: Hello, Updated version: http://cr.openjdk.java.net/~serb/8027696/webrev.01/ Dates and spaces were fixed. Thanks Sergey, I sampled a few and they look correct. The only thing is that I'm not sure about is test/Makefile, it's just not clear whether whi

Re: [8] Review Request: 8027696 Incorrect copyright header in the tests

2013-11-07 Thread Sergey Bylokhov
On 07.11.2013 15:22, Alan Bateman wrote: Thanks Sergey, I sampled a few and they look correct. The only thing is that I'm not sure about is test/Makefile, it's just not clear whether which header this should have. I can skip it this time. -- Best regards, Sergey.

Re: [8] Review Request: 8027696 Incorrect copyright header in the tests

2013-11-11 Thread Sergey Bylokhov
Hello. I'll push the changes, if there are no new objections. On 07.11.2013 17:46, Sergey Bylokhov wrote: On 07.11.2013 15:22, Alan Bateman wrote: Thanks Sergey, I sampled a few and they look correct. The only thing is that I'm not sure about is test/Makefile, it's just not clear whether whic