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

Fix macos notarise #1889

Merged
merged 2 commits into from
Jul 28, 2022
Merged

Fix macos notarise #1889

merged 2 commits into from
Jul 28, 2022

Conversation

tonymorony
Copy link

No description provided.

@tonymorony tonymorony requested a review from smk762 July 27, 2022 17:59
@tonymorony
Copy link
Author

@smk762 was a deep dive lol, notarisations should be fixed now for both dmg and installer (it looks like apple added team indeficator parameter ASC_PUBLIC_ID as mandatory just recently and macos_notarize.sh not updated nor in our copypaste, nor in original tool https://github.com/drud/signing_tools/blob/master/macos_notarize.sh

Copy link
Collaborator

@smk762 smk762 left a comment

Choose a reason for hiding this comment

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

Nice one )
Working as expected on mac here with latest CI binaries.

@smk762 smk762 requested review from a user, Canialon and SirSevenG July 28, 2022 06:29
@tonymorony tonymorony merged commit bfc1972 into dev Jul 28, 2022
@tonymorony tonymorony deleted the fix_macos_notarise branch July 28, 2022 09:51
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