-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
electron-builder not generating app-update.yml for Linux #6322
Comments
Autoupdate only supports AppImage, so your
|
@mmaietta There is a problem here though. The problem is that
While these are not fatal, it is annoying to pollute the log file with them. Is there something I can do in the code to figure out that the application is running from a non-supported installer platform? If not, the auto-updater should probably figure this out somehow and make the calls no-ops. |
App-update.yml is not genereate on Windows also. I do not know, no error message. |
@panther7 app-update.yaml should be generated when building the NSIS installer (and are using the Are you saying that your My previous comment was a runtime, rather than a build-time issue. At runtime, I added a check like this to my code to eliminate the errors from our log file.
|
@robertpatrick Yes, you have right, but, why does not works autoupdate with archive like zip? |
I still dont understand, why is not support create "app-update.yml" for each situation. Now, i must "manualy" create app-update.yml with afterPack callback :-/ |
@panther7 i think the better question is why electron-builder thinks you need this file for unsupported installer types |
Suggested fix for electron-userland#6322 (not tested yet).
@rpatrick00 maybe we can do something like this? electron-builder/packages/electron-updater/src/AppUpdater.ts Lines 252 to 258 in b01d522
Replace with
A few changes elsewhere are needed, such as My only pickle right now is: How do we detect it's an NSIS-installer app and not MSI? Do you know if there are any env vars we can check for at runtime? |
Wait, it doesn't need that extra check. It already checks in AppImageAdapter electron-builder/packages/electron-updater/src/AppImageUpdater.ts Lines 18 to 28 in b01d522
Are you guys using |
@mmaietta in our app, we are using checkForUpdates(). The app was working fine from an RPM installation but the checkForUpdates() call resulted in a few error messages being written to the log. I wrapped the call to checkForUpdates to not call it is the platform is Linux and the APPIMAGE environment variable is not set to prevent these error messages from being written to the log... |
… by `checkForUpdatesAndNotify`. Fixes: #6322
The |
Huge thanks for fixing this issue! Was a big deal with appx in windows store |
After this change I cannot force the application to check for Updates if it's not packaged, even if I use checkForUpdates and place a dev-app-update.yml in the root folder of the app next to the package.json. Is this intended? |
Yes, that seems to be intended from previous business logic electron-builder/packages/electron-updater/src/AppUpdater.ts Lines 256 to 262 in 86e6d15
|
here is my package.json file
When I build my Linux app using
sudo yarn build-linux
and opens the app, auto update doesn't work. When I look at the logs, it saysIt seems app-update.yml doesn't get generated. Can anybody guide me what I am doing wrong?
The text was updated successfully, but these errors were encountered: