Hi Andy,

http://cr.openjdk.java.net/~herrick/8237490/webrev.05/src/jdk.incubator.jpackage/macosx/classes/jdk/incubator/jpackage/internal/MacAppImageBuilder.java.frames.html
Line 819,857,902 - Looks like temp debug log message. Remove it or align with rest of code.

http://cr.openjdk.java.net/~herrick/8237490/webrev.05/src/jdk.incubator.jpackage/macosx/classes/jdk/incubator/jpackage/internal/resources/MacResources.properties.frames.html
Line 70 - Capital F.

Since we added "--timestamp" and  "--options runtime" to codesign, will it work with older Xcode and macOS we planning to support?

Do we need any adjustments to signing tests we have?

Otherwise looks fine.

Thanks,
Alexander

On 3/27/20 12:35 PM, Andy Herrick wrote:
Please review the fix to issue [1] at [2].

This change enables notarization on Mac for dmg images and app-image zip files.

/Andy

[1]: https://bugs.openjdk.java.net/browse/JDK-8237490

[2]: http://cr.openjdk.java.net/~herrick/8237490


Reply via email to