RFR(XXXXXXXS): 8161034 Incorrect GPL header in jdk/src/java.base/windows/native/libjava/jni_util_md.c
Hello, src/java.base/windows/native/libjava/jni_util_md.c file contains incorrect GPL header which fails make/scripts/lic_check.sh script check. Failure is caused by missing comma after modification years. Please, help to review its addition: http://cr.openjdk.java.net/~fparain/8161034/webrev.00/ lic_check.sh passes after this modification: ### Checking copyright notice in the file: jdk/src/java.base/windows/native/libjava/jni_util_md.c ### SUCCESS: The license header for /export/fparain/WORK/JDK9/8161034/hs/jdk/src/java.base/windows/native/libjava/jni_util_md.c has been verified. ### Thank you, Fred
On 18/07/2016 14:48, frederic parain wrote:
Hello,
src/java.base/windows/native/libjava/jni_util_md.c file contains incorrect GPL header which fails make/scripts/lic_check.sh script check.
Failure is caused by missing comma after modification years. Please, help to review its addition: http://cr.openjdk.java.net/~fparain/8161034/webrev.00/
lic_check.sh passes after this modification: ### Checking copyright notice in the file: jdk/src/java.base/windows/native/libjava/jni_util_md.c ### SUCCESS: The license header for /export/fparain/WORK/JDK9/8161034/hs/jdk/src/java.base/windows/native/libjava/jni_util_md.c has been verified. ### Looks fine.
Thanks for the review Alan! Fred On 18/07/2016 10:39, Alan Bateman wrote:
On 18/07/2016 14:48, frederic parain wrote:
Hello,
src/java.base/windows/native/libjava/jni_util_md.c file contains incorrect GPL header which fails make/scripts/lic_check.sh script check.
Failure is caused by missing comma after modification years. Please, help to review its addition: http://cr.openjdk.java.net/~fparain/8161034/webrev.00/
lic_check.sh passes after this modification: ### Checking copyright notice in the file: jdk/src/java.base/windows/native/libjava/jni_util_md.c ### SUCCESS: The license header for /export/fparain/WORK/JDK9/8161034/hs/jdk/src/java.base/windows/native/libjava/jni_util_md.c has been verified. ### Looks fine.
participants (2)
-
Alan Bateman
-
frederic parain