Skip to content

Conversation

@jkennedyvz
Copy link
Contributor

@jkennedyvz jkennedyvz commented Jun 1, 2024

Related to #251

Updates the CI pipeline to use notarytool for macOS notarization and removes references to Gon.

  • Removes the section that installs and uses Gon for macOS notarization in the .github/workflows/ci.yaml file.
  • Adds steps to install notarytool, prepare the application for notarization, submit it for notarization, and staple the notarization ticket to the application in the macOS build steps.
  • Utilizes environment variables for Apple ID, Team ID, and an app-specific password to authenticate with Apple's notarization service.

For more details, open the Copilot Workspace session.

@jkennedyvz jkennedyvz requested a review from jrozner as a code owner June 1, 2024 18:25
@jrozner
Copy link
Member

jrozner commented Jun 1, 2024

I ran into the libbrotli already signed issue in the Qt upgrade branch. I think homebrew must have started signing stuff that it installs or something and so we need to remove the signature in order to allow us to sign it. You can see the fix in src/CMakeLists.txt in #248. I'm not sure if there is a better way to do it or not but that did fix it.

@jrozner
Copy link
Member

jrozner commented Jun 1, 2024

Sorry, you need to take just that one line and unmerge the rest of this. There are other problems that that branch introduces that are less trivial to fix.

@jkennedyvz jkennedyvz force-pushed the jkennedyvz/update-notarytool branch 3 times, most recently from 066ca48 to 3d9ccc9 Compare June 1, 2024 19:46
@jkennedyvz jkennedyvz force-pushed the jkennedyvz/update-notarytool branch from 3d9ccc9 to 2df4fb3 Compare June 1, 2024 19:52
@jkennedyvz
Copy link
Contributor Author

Everything seems to work now!

@jrozner jrozner merged commit 8098162 into main Jun 1, 2024
@jrozner jrozner deleted the jkennedyvz/update-notarytool branch June 1, 2024 20:05
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.

3 participants