Skip to content
This repository has been archived by the owner on Dec 28, 2021. It is now read-only.

Enable build signing for macOS and Windows binaries. #1366

Merged

Conversation

MichaelMauderer
Copy link
Contributor

@MichaelMauderer MichaelMauderer commented Mar 24, 2021

[ci build]

Pull Request Description

Enables signing of our build artefacts for windows and macOS. For macOS, we also notarize the project.

Important Notes

  • a lot of the extra signing logic here will move to the engine once engine builds are signed and notarized.
  • enables [ci build] tag to be used in the PR body instead of the commit message.

Checklist

Please include the following checklist in your PR:

  • The CHANGELOG.md was updated with the changes introduced in this PR.
  • The documentation has been updated if necessary.
  • All code conforms to the Rust style guide.
  • All code has automatic tests where possible.
  • All code has been profiled where possible.
  • All code has been manually tested in the IDE.
  • All code has been manually tested in the "debug/interface" scene.
  • All code has been manually tested by the PR owner against our test scenarios.
  • All code has been manually tested by at least one reviewer against our test scenarios.

@MichaelMauderer MichaelMauderer changed the title chore: Enable build signing for macOS and Windows binaries. Enable build signing for macOS and Windows binaries. Mar 24, 2021
@MichaelMauderer MichaelMauderer force-pushed the wip/mm/ide-1135-sign-windows-and-macos-binaries.- branch 18 times, most recently from 9e433da to ac53ec7 Compare March 29, 2021 10:35
@MichaelMauderer MichaelMauderer self-assigned this Apr 6, 2021
@MichaelMauderer MichaelMauderer force-pushed the wip/mm/ide-1135-sign-windows-and-macos-binaries.- branch 4 times, most recently from 92978f5 to a156b47 Compare April 8, 2021 10:03
@MichaelMauderer MichaelMauderer force-pushed the wip/mm/ide-1135-sign-windows-and-macos-binaries.- branch from a156b47 to 4a4d1ae Compare April 8, 2021 10:04
@MichaelMauderer MichaelMauderer marked this pull request as ready for review April 8, 2021 14:35
@MichaelMauderer MichaelMauderer added the Category: Build System The build scripts and CI. label Apr 8, 2021
Copy link
Collaborator

@farmaazon farmaazon left a comment

Choose a reason for hiding this comment

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

Only minor comments, however I can test only Windows (which AFAIK was already tested by @mwu-tow ).

CHANGELOG.md Outdated Show resolved Hide resolved
src/js/lib/client/tasks/signArchives.js Outdated Show resolved Hide resolved
src/js/lib/client/tasks/signArchives.js Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
src/js/lib/client/entitlements.mac.plist Show resolved Hide resolved
src/js/lib/client/package.js Outdated Show resolved Hide resolved
src/js/lib/client/package.js Show resolved Hide resolved
src/js/lib/client/tasks/notarize.js Show resolved Hide resolved
src/js/lib/client/tasks/signArchives.js Outdated Show resolved Hide resolved
src/js/lib/client/tasks/signArchives.js Outdated Show resolved Hide resolved
src/js/lib/client/tasks/signArchives.js Outdated Show resolved Hide resolved
src/js/lib/client/tasks/signArchives.js Outdated Show resolved Hide resolved
src/js/lib/client/tasks/signArchives.js Outdated Show resolved Hide resolved
Copy link
Member

@wdanilo wdanilo left a comment

Choose a reason for hiding this comment

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

I love the new comments! <3

src/js/lib/client/entitlements.mac.plist Outdated Show resolved Hide resolved
}
// We need to manually re-sign our build artifacts before notarisation.
// See the script for more information.
console.log(" • Extra Signing.")
Copy link
Member

Choose a reason for hiding this comment

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

I mean in the console - it should be named in such a way that when we see it during building in console, we understand what is being done then.

@MichaelMauderer MichaelMauderer merged commit 99de1d7 into develop Jun 10, 2021
@MichaelMauderer MichaelMauderer deleted the wip/mm/ide-1135-sign-windows-and-macos-binaries.- branch July 7, 2021 11:57
mwu-tow pushed a commit to enso-org/enso that referenced this pull request Oct 30, 2021
mwu-tow pushed a commit to enso-org/enso that referenced this pull request Oct 30, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Category: Build System The build scripts and CI. Difficulty: Hard Significant prior knowledge requried Priority: High Should be scheduled as soon as possible Type: Enhancement An enhancement to the current state of Enso IDE
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants