On Apr 26, 2020, at 10:18 PM, Igor Ignatev <igor.ignatyev@oracle.com> wrote:

LGTM

— Igor

...

On Apr 26, 2020, at 7:14 PM, Kim Barrett <kim.barrett@oracle.com> wrote:

Please review this fix for a missing comma in the copyright header for src/jdk.incubator.jpackage/share/classes/jdk/incubator/jpackage/internal/DottedVersion.java.

CR: https://bugs.openjdk.java.net/browse/JDK-8243587

Webrev: https://cr.openjdk.java.net/~kbarrett/8243587/webrev/

Testing: Tested locally (linux-x64): make validate-headers fails without the change, succeeds with it.

Thanks