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

deps: Update electron-builder to 24.6.5 #737

Closed
wants to merge 3 commits into from

Conversation

DeeDeeG
Copy link
Member

@DeeDeeG DeeDeeG commented Sep 22, 2023

Identify the Bug

Pulsar's macOS binaries from GitHub Actions are either not signed, or signed in such a way that the Gatekeeper popup shows:

“Pulsar” cannot be opened because the developer cannot be verified.

This PR is trying something as a bit of a long shot that might address the "not properly signed" scenario. (The "completely unsigned" scenario is due to an issue with the workflow file, and appears to be a separate issue.)

Description of the Change

Bump our electron-builder dependency to 24.6.3.

Maybe this will fix macOS signing/notarizing in GitHub Actions???

Need to run a PR and see if it works to find out, since this is, for our intents and purposes, currently a CI-only issue.

Alternate Designs

This is just a check-and-see "maybe fix". No specific alternatives were considered, besides perhaps locally trying to reproduce the "signing/notarizing didn't work" issue. That remains an option.

Possible Drawbacks

See the release notes, I haven't read through them exhaustively, but nothing particularly stood out to me...

https://github.com/electron-userland/electron-builder/releases

I chose the latest one not marked "Pre-Release", so this should be a relatively stable and well-tested version, I hope?

Verification Process

  • CI should pass.
  • Downloaded macOS assets from this PR's GitHub Actions "build Pulsar binaries" workflow should be fully properly signed. They should open without presenting the "cannot be verified" popup.

Release Notes

(Tentative, pending testing): "Fixed app signing for macOS in GitHub Actions CI"

Maybe this will fix macOS signing/notarizing in GitHub Actions???
This is for newer electron-builder. This needs to be set, or else the
latest electron-builder won't attempt to sign/notarize for
Pull Requests.

We want to at least try this once, to try signing in this PR, and see
if the latest electron-builder version fixes the signing/notarizing
issue we have been seeing lately from our binaries built on
GitHub Actions?
@DeeDeeG DeeDeeG force-pushed the bump-electron-builder-24.x branch from 72e9b46 to a12ad63 Compare September 24, 2023 17:19
@DeeDeeG
Copy link
Member Author

DeeDeeG commented Sep 25, 2023

Ze dependency bumps, zey do nossing!

(https://knowyourmeme.com/memes/the-goggles-do-nothing)

@DeeDeeG DeeDeeG closed this Sep 25, 2023
@DeeDeeG DeeDeeG reopened this Sep 25, 2023
This version is still considered a "beta" or "prerelease" version,
but it bumps some dependencies related to macOS signing and
notarizing, so it's worth a shot!
@DeeDeeG DeeDeeG changed the title deps: Update electron-builder to 24.6.3 deps: Update electron-builder to 24.6.5 Sep 25, 2023
@JonathanDoughty
Copy link

JonathanDoughty commented Sep 26, 2023

Given the information on mastodon and Ars Technica you might want to target Electron v24.8.3 or later. Though I am unsure about the relationship with electron-builder it appears that released Pulsar is using Electron 18.3.5

@DeeDeeG
Copy link
Member Author

DeeDeeG commented Sep 29, 2023

@JonathanDoughty I've seen that info, and yeah, just looking at which Electron versions, it would appear to affect Pulsar, unfortunately.

Getting the editor on newer versions of Electron was a main focus around Atom toward the end, even since before we forked from it to make Pulsar, and likewise since forking it's something we in pulsar-edit org want to do as soon as we feasibly can. And we do have efforts taking place to jump to the very latest Electron version. There's been discussion on our Discord server of putting a beta version out that does that, brought up again in the past few days, but it's been a big effort that's been built toward for a long time. (Credit mostly to @mauricioszabo for these efforts so far.) I wouldn't commit to any specific timeline other than "as soon as we can make sure it doesn't break (a lot of) things." (For example, last I heard the github package that handles CLI git integration and GitHub PR integration into Pulsar was not working with newer Electron versions yet.)

We are unfortunately still on Electron 12 at the time I'm writing this. And of course we want to get on a newer version as soon as feasible, for security reasons.

@DeeDeeG
Copy link
Member Author

DeeDeeG commented Sep 29, 2023

I'm closing this PR, since this was meant to update our build tools and fix code signing, not bump the Electron version (that could be posted as a separate issue).

I don't mind at all if there's an issue or Discussion opened about the Electron version being out of date, if anyone wants there to be such a dedicated issue or Discussion. (If that's undesirable and you have just something brief to add, comments can still be posted to closed PRs.)

(This PR isn't needed anymore since #743 took care of the code signing situation. I'd like to close this one to clean up the open PRs list.)

@DeeDeeG DeeDeeG closed this Sep 29, 2023
@DeeDeeG DeeDeeG deleted the bump-electron-builder-24.x branch November 16, 2023 03:40
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