Skip to content

Force notarization process to take place configuring Gon zip output #580

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Feb 12, 2020

Conversation

rsora
Copy link
Contributor

@rsora rsora commented Feb 12, 2020

In order to force the notarization process to take place, we need to ask gon to produce a .zip file as output.

As explained here #578 / #578 (comment) the gon.config.hcl "zip" section is not mandatory di per se', but not having an "output" section at all will not start the notarization submission process for the binary.

For our release process the generated .zip file has no use, as we repackage the signed binary that gon replaces in-place. The .zip is functional for the notary service submission only

Thanks to @zmoog for patiently explaining the whole process to me countless times 🙇‍♂️

@rsora rsora added this to the 0.8.0 milestone Feb 12, 2020
@zmoog
Copy link
Contributor

zmoog commented Feb 12, 2020

Yeah, it looks like the altool, that gon is orchestrating behind the scenes, require the binary to be packaged in a zip file, as described in this Customizing the Notarization Workflow article.

So, LGTM.

@zmoog zmoog self-requested a review February 12, 2020 11:29
Copy link
Contributor

@zmoog zmoog left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@rsora rsora merged commit 4ed7183 into master Feb 12, 2020
@rsora rsora deleted the rsora/notarize-zip branch February 12, 2020 11:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants