-
-
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
NSIS Autoupdater Fails to update #945
Comments
Does it occurred occasionally or always? |
Please specify electron-auto-updater version. |
Where do I get the electron-auto-updater version? And it does occur on every try I made |
electron-auto-updater is a dependency listed in your package.json. |
It says ^0.5.0 |
Please use electron-builder 10.2.0 and electron-auto-updater |
Ok my deps seem to be messed up. I will try to fix that. thanks for help |
With yarn it works :) |
Hello, I have the same error. But the different is , the desktop app can update to version 1.0.2 correctly at the first install time with version 1.0.1. But appear 'spawn UNKNOWN' error, when update to version 1.0.3 with version 1.0.2. How did you fix it, and what's yarn? Tks! |
@moJiXiang Please ensure that you use latest version of electron-auto-updater. |
@develar Thank you. electron-builder ^10.9.2 and electron-auto-updater ^0.9.2. |
@moJiXiang Sorry for confusion, but latest electron-auto-updater requires latest electron-builder (11.2.1, #925 (comment)). |
@develar Oh, sorry. Try again. |
@develar Sorry, same error again. What's yarn, MTRNord use yarn and it worked. |
Yarn is a packages alternativ to npm. It is a client tool |
@MTRNord Yarn can resolve 'spawn UNKNOWN' error? Why? |
@moJiXiang yarn did fix some version problems for me. npm didn't provide the newest version on my PC. Yarn did |
@moJiXiang What version of electron-auto-updater your app 1.0.2 uses? |
@develar All version use electron-auto-updater 0.9.2, I just create a simple demo which use one step nsis and electron-auto-updater 0.10.1 to rule out some unlimited bug. But same error again. |
@moJiXiang Latest electron-auto-updater version is |
@develar I just create a simple demo which use one step nsis and electron-auto-updater 0.10.1 to rule out some unlimited bug. But same error again. |
@develar I found error on the install step. Code in the (0, (_child_process || _load_child_process()).spawn)(setupPath, args, {
detached: true,
stdio: "ignore"
}).unref(); It like child_process spaw unknown |
@develar Hello, I test many times. But I can't ensure it's electron-auto-updater bug, or just my mistake. I'll find someone help me build a windows app to test. |
@moJiXiang I will do manual testing of latest electron-auto-updater today or tomorrow. |
@develar Hi there, I have been having this exact same problem... If I run my installed app normally it throws the error when trying to install the update after download, but if I run the app as administrator it downloads and updates fine. Unfortunately this is not a solution for me, the user as to be able to update the app just by using it normally. Is there a way to force the app to run as administrator (as a last case scenario)? I have the latest version of electron-builder and electron-auto-updater, updated them today. Ps: Maybe this has something to do with the app being from an Unknown publisher? |
auto-update must show the same UI (except license page). It seems, currently it is broken. Sorry for inconvenience. 1.0.0 will be published this month and we will avoid suc regressions in the future. I am working on fix. |
@moJiXiang Fixed in 12.0.2 as #1129 |
I got the same question and I used electron-builder ^12.3.1,electron-updater ^1.16.0 |
I use Github for updates and download works but on install it fails.
My updater code is (imports are simplified):
Following errorLog comes after starting the install:
The text was updated successfully, but these errors were encountered: