RFR: JDK-8237490: [macos] Add support notarizing jpackage app-image and dmg
andy.herrick at oracle.com
Sat Mar 28 18:49:53 UTC 2020
On 3/28/2020 9:43 AM, Andy Herrick wrote:
> On 3/27/2020 5:18 PM, Alexander Matveev wrote:
>> Hi Andy,
>> Line 819,857,902 - Looks like temp debug log message. Remove it or
>> align with rest of code.
> I will fix this.
>> Line 70 - Capital F.
> and this
>> Since we added "--timestamp" and "--options runtime" to codesign,
>> will it work with older Xcode and macOS we planning to support?
> not sure - may need some discussion of what we support and possible
> conditional code here.
The "--timestamp" argument existed in earlier versions of XCode, but the
option "runtime" is just since XCode 10.14.0, however the man page for
codesign specifically says that macOS versions older than 10.14.0 ignore
the presence of this flag in the code signature.
>> Do we need any adjustments to signing tests we have?
> The existing tests pass, but this is not unexpected (and really means
> nothing) since the signing tests are all skipped unless specific test
> certs are installed on target machine.
> We need further discussion how one is expected to provision a machine
> to run these tests.
>> Otherwise looks fine.
>> On 3/27/20 12:35 PM, Andy Herrick wrote:
>>> Please review the fix to issue  at .
>>> This change enables notarization on Mac for dmg images and app-image
>>> zip files.
>>> : https://bugs.openjdk.java.net/browse/JDK-8237490
>>> : http://cr.openjdk.java.net/~herrick/8237490
More information about the core-libs-dev