Skip to content
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

Replace mitchellh/go used for macOS notarization #30

Merged
merged 1 commit into from
Nov 28, 2023
Merged

Replace mitchellh/go used for macOS notarization #30

merged 1 commit into from
Nov 28, 2023

Conversation

umbynos
Copy link
Contributor

@umbynos umbynos commented Nov 28, 2023

Description

An excellent tool named gon was used to perform the notarization.
the latest stable release of gon uses the altool command-line utility for notarization:
https://github.com/mitchellh/gon/blob/v0.2.5/notarize/upload.go#L41

Using altool for notarization is now deprecated by Apple and support for notarization via this tool is scheduled to be disabled 2023-11-01:
https://developer.apple.com/news/?id=y5mjxqmn

Furthermore, gon was archived.

I switched to https://github.com/Bearer/gon that has included mitchellh/gon#72, hoping the maintainers will take the responsibility of maintaining the project.
I added also AC_PROVIDER env var which is mandatory with the new notarytool.

See arduino/tooling-project-assets#359

@umbynos umbynos added type: enhancement Proposed improvement topic: infrastructure Related to project infrastructure labels Nov 28, 2023
@umbynos umbynos self-assigned this Nov 28, 2023
@umbynos umbynos merged commit b85bd88 into main Nov 28, 2023
5 checks passed
@umbynos umbynos added the conclusion: resolved Issue was resolved label Nov 28, 2023
@umbynos umbynos deleted the gon-bye branch October 8, 2024 15:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
conclusion: resolved Issue was resolved topic: infrastructure Related to project infrastructure type: enhancement Proposed improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants