-
Notifications
You must be signed in to change notification settings - Fork 856
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
Upgrade electron-builder #749
Conversation
@yuya-oc Looks like only the Zip packages were uploaded on CircleCI: |
b41a5df
to
7b80586
Compare
@jasonblais Thanks to let me know. Artifacts name had been changed by electron-builder. Here is fixed version. |
@yuya-oc I'm getting an error when trying to launch the app on Mac ( |
@jasonblais Reproduced also on local build. Probably it's a bug of electron-builder. (similar issue: electron-userland/electron-builder#784) But possibly it's good time to move to zip (it works). And it's required for auto-updater. |
@jasonblais I think there are nothing other than filename (of course we should test though). For mac, dmg and zip are almost same, but they have different purpose in auto-updater context.
|
@yuya-oc So would auto-updates work for .dmg? Or would the user download .dmg initially (if desired), but then .zip files for updates? |
As for using tar.gz or zip files, I'm okay moving to .zip files if there are no changes to the user. Especially since we'll be changing to it anyways soon. |
auto-updater would work for both of zip and dmg. But they check whether .zip exists on the server. So their difference is just file format (and some extra pictures for appearance of dmg). I'll prepare zip version. |
Tested on Ubuntu 16.04. No issue found, and icons are correctly included. |
Thanks all! @yuya-oc we should be good to merge this one. |
Before submitting, please confirm you've
npm run lint:js
for proper code formattingPlease provide the following information:
Summary
Upgrade electron-builder.
It has an issue about Linux icon. As a workaround, macOS icon is also used for Linux in this PR.
Issue link
#702
Test Cases
Download artifacts. Then, check artifacts name, package name, publisher name and icons.
Additional Notes
https://circleci.com/gh/yuya-oc/desktop/736#artifacts