RE: RFR(XS): 8213151: [AIX] Some class library files are missing the Classpath exception
Hi Volker, looks good (not a Reviewer however) ! Maybe you should also adjust the Copyright year info in the files you touched . Best regards, Matthias -----Original Message----- Hi, can I please have a review for the following tiny change which fixes the license header on a few AIX-specific files: http://cr.openjdk.java.net/~simonis/webrevs/2018/8213151/ https://bugs.openjdk.java.net/browse/JDK-8213151 These files have been wrongly integrated (without the Classpath Exception in their license header) from the very beginning of the AIX port. The fix is trivial - just add the CPE to these files similar to the corresponding sibling files. Thank you and best regards, Volker
Looks fine, Reviewed. Thanks, Roger On 10/30/2018 11:02 AM, Baesken, Matthias wrote:
Hi Volker, looks good (not a Reviewer however) ! Maybe you should also adjust the Copyright year info in the files you touched .
Best regards, Matthias
-----Original Message-----
Hi,
can I please have a review for the following tiny change which fixes the license header on a few AIX-specific files:
http://cr.openjdk.java.net/~simonis/webrevs/2018/8213151/ https://bugs.openjdk.java.net/browse/JDK-8213151
These files have been wrongly integrated (without the Classpath Exception in their license header) from the very beginning of the AIX port. The fix is trivial - just add the CPE to these files similar to the corresponding sibling files.
Thank you and best regards, Volker
Thanks for the quick reviews! Volker On Tue, Oct 30, 2018 at 4:35 PM Roger Riggs <Roger.Riggs@oracle.com> wrote:
Looks fine, Reviewed.
Thanks, Roger
On 10/30/2018 11:02 AM, Baesken, Matthias wrote:
Hi Volker, looks good (not a Reviewer however) ! Maybe you should also adjust the Copyright year info in the files you touched .
Best regards, Matthias
-----Original Message-----
Hi,
can I please have a review for the following tiny change which fixes the license header on a few AIX-specific files:
http://cr.openjdk.java.net/~simonis/webrevs/2018/8213151/ https://bugs.openjdk.java.net/browse/JDK-8213151
These files have been wrongly integrated (without the Classpath Exception in their license header) from the very beginning of the AIX port. The fix is trivial - just add the CPE to these files similar to the corresponding sibling files.
Thank you and best regards, Volker
participants (3)
-
Baesken, Matthias
-
Roger Riggs
-
Volker Simonis