-
-
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
On Windows, running app not quit when uninstalling the app but uninstallation just completed #7493
Comments
What electron-updater version are you using? Was this working previously? |
the version of electron-updater is at: ^5.3.0. Yeah it worked previously, Recently we add customUninstall nsh script to delete Registry Keys, but it doesn't work even this part is reverted. |
do you have any other insights? |
I don't know enough about |
hmm,, I tried removing that customized nsis stuff, and test in a clean Windows VM, but still no luck. Downgrade to electron-builder 23.3.0 is also not working. I've another app uses 22.11.11, it works correctly. UPDATE: |
Is there any way to enable logs on windows? I checked some issues e.g. brimdata/zui#2713 but seems not solution.. Maybe I should upgrade to 24.1.2 to have a try? 24.1.2 still not working either. |
You'll need to provide your own custom nsis binary that supports debug logging. (I tried to build it in directly to electron-builder, but 3.0.4.2 nsis was throwing off alerts from antivirus programs, so we are stuck with 3.0.4.1
Quick link to the interface below:
|
Thanks @mmaietta I got> "Error: LogSet specified, NSIS_CONFIG_LOG not defined." when I set {
url: 'https://github.com/electron-userland/electron-builder-binaries/releases/download',
debugLogging: true,
checksum:
'VKMiizYdmNdJOWpRGz4trl4lD++BvYP2irAXpMilheUP0pc93iKlWAoP843Vlraj8YG19CVn0j+dCo/hURz9+Q==',
version: '3.0.4.1',
} Tried "nsis-3.0.8-log", and now it can output logs. I saw it tries to remove the Exe file directly, but no logs say it is checking the app running or not.
BTW, do you know why 22.11.11 works? I checked the changes history between 22.11.11 ---> 23.6.0, only some MRs merged:
Those MRs changed the "uninstall.nsh", but not sure which one. |
https://github.com/electron-userland/electron-builder/pull/6024/files |
Here is the log file, would you pls help a look? Thanks. The error I saw is:
Looks like the app not quit before deleting.. |
@mmaietta would you pls help check the above logs? thanks. |
In all honesty, I have no idea how nsis works or am able to debug installers. I did take a look at the |
@mmaietta I found something that, If I changed the product name to another one, then it works. E.g. If it is named "Desktop Plugin" it works, but if changed to name ("RingCentral Desktop Plugin"), it is not working. Is there a bug in the process name comparing? Since if I changed to electron-builder: 22.11.11, both work. Those names work:
Those names do not work:
|
@indutny @lwintermelon do you have any idea about this issue? Would you please help check? Since I saw last last time you did some changes on uninstaller part. Appricated. Thanks! |
@kafka-yu could you provide the |
Sorry for the late reply. The issus is very similar to what I fixed on #6100, maybe you can check this PR and try to fix the issue. |
thanks for your reply! @lwintermelon I checked but have no idea about the fixing solution, could you pls give some hints about this part? Thank you! |
This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days. |
This issue was closed because it has been stalled for 5 days with no activity. |
I am also facing the same issue after I changed the product name. Electron-Builder Version : 23.6.0 |
I am having this issue. I uninstalled it, and i kept getting a popup saying, "electron.exe System error" something about dll, and how I had to reinstall the app to fix the issue. But seeing as I'd get this popup every time I turned on my PC. I knew it was running in the background causing the popups. For me what fixed it, I just opened task manager and found that electron was somehow still running in the background, I just selected it after closing the popup, and deleted it from the task manager menu. Hopefully this helps someone else. |
Hopefully you found the issue, did you try to open task manager to see if it was somehow running in the background? |
Hi team,
we find a strange issue: When uninstalling, the running app is not able to quit automatically. The uninstallation is completed fast. And the files are deleted except the files that are app running used.
The package below config to pack the app, and with a custom NSIS script to do "customInstall" and "customUninstall" steps.
The installer.nsh file content:
Does anyone know the reason?
The text was updated successfully, but these errors were encountered: