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

feat: add Electron branding options #5727

Merged

Conversation

samuelmaddock
Copy link
Contributor

The Electron project contains a BRANDING.json file which will modify the resulting executable file names. A project I'm working on does exactly this in a fork of Electron.

Exposing these branding options allows Electron forks to build properly if branding has changed.

Copy link
Collaborator

@mmaietta mmaietta 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 this and had the exact same issue to work around, I also didn't realize that the BRANDING.json was available for changing exec names though. We had it manually patched in our fork. 😅

Approved.

Will wait a little before merging in case anyone else would like to chime in with feedback

@mmaietta mmaietta merged commit 0de5f1f into electron-userland:master Mar 29, 2021
@samuelmaddock samuelmaddock deleted the feat/electron-branding branch April 6, 2021 16:51
@samuelmaddock
Copy link
Contributor Author

@mmaietta sorry to bother, but is there any rough estimate on a schedule for publishing recent changes?

@mmaietta
Copy link
Collaborator

Hi @samuelmaddock, apologies on the delay. I don't have the right access to publish builds, but we just did release one! Please try 22.11.3

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.

None yet

2 participants