Skip to content
This repository was archived by the owner on Jun 24, 2025. It is now read-only.

Enable MacOS code signing and notarization in GitHub Actions#1267

Merged
eliandoran merged 6 commits intoTriliumNext:developfrom
perfectra1n:develop
Feb 24, 2025
Merged

Enable MacOS code signing and notarization in GitHub Actions#1267
eliandoran merged 6 commits intoTriliumNext:developfrom
perfectra1n:develop

Conversation

@perfectra1n
Copy link
Copy Markdown
Member

@perfectra1n perfectra1n commented Feb 24, 2025

? path.join(buildPath, `${APP_NAME}.app`, "Contents", "Resources", baseName)
: path.join(buildPath, "resources", baseName);

// prettier-ignore
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

could you add that comment back in above destPath please?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I believe I put it back where you asked, now 👀

@perfectra1n
Copy link
Copy Markdown
Member Author

perfectra1n commented Feb 24, 2025

That afterComplete hook probably cost me 6 hours of my life, wondering why signing and notarization didn't work after Electron said it did everything 😭

(because it moved files around in the .app after it had already been signed, broke the seal on the signature)

So we'll want to keep that in mind if we ever want to sign Windows, that we'll need to modify that afterComplete hook to also not change Windows' files around in any build artifacts after the signature has already been competed...

@eliandoran eliandoran merged commit 4240af6 into TriliumNext:develop Feb 24, 2025
1 check passed
@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Jun 21, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

(Bug report) Damaged macOS binary build

3 participants