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

fix(app-builder-lib): macOS packager uses static icon name #6308

Merged
merged 3 commits into from Oct 3, 2021

Conversation

sr258
Copy link
Contributor

@sr258 sr258 commented Sep 30, 2021

The filename of the app icon in macOS is now always 'icon.icns' instead of a derivate of the product name. The reason for this change is that macOS doesn't display icons with non-ASCII characters in their names, which is quite possible in languages other than English.

This fixes one of the issues described here #6282

@changeset-bot
Copy link

changeset-bot bot commented Sep 30, 2021

🦋 Changeset detected

Latest commit: 8fc206c

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 8 packages
Name Type
app-builder-lib Patch
dmg-builder Patch
electron-builder-squirrel-windows Patch
electron-builder Patch
electron-forge-maker-appimage Patch
electron-forge-maker-nsis-web Patch
electron-forge-maker-nsis Patch
electron-forge-maker-snap Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@netlify
Copy link

netlify bot commented Sep 30, 2021

✔️ Deploy Preview for jolly-roentgen-9c9aba ready!

🔨 Explore the source changes: 8fc206c

🔍 Inspect the deploy log: https://app.netlify.com/sites/jolly-roentgen-9c9aba/deploys/6156a550d56b9f0007eebb1e

😎 Browse the preview: https://deploy-preview-6308--jolly-roentgen-9c9aba.netlify.app

@mmaietta
Copy link
Collaborator

mmaietta commented Oct 1, 2021

Can you try rebasing this off latest master? Not sure why one of the tests are failing

@sr258
Copy link
Contributor Author

sr258 commented Oct 1, 2021

@mmaietta I rebased this and fixed the test snapshots as far as possible. Unfortunately, I can't really run the tests on my dev machine, as I use an M1 processor, which causes lots of tests to fail even though they shouldn't (see #6310). I hope the CI pipeline runs through this time.

@mmaietta mmaietta merged commit fce1a1f into electron-userland:master Oct 3, 2021
@github-actions github-actions bot mentioned this pull request Oct 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Broken icon and dmg layout with non-ASCII characters in productName on macOS
2 participants