Xpra: Ticket #495: Windows: Uninstaller will run and "uninstall" while Xpra is still running

In Windows, the uninstaller from:

Control Panel -> Programs, etc. ->Uninstall a Program

will open the uninstaller, which goes through the process of uninstalling Xpra. Before finishing, a message pops up saying not everything could be deleted.

After closing it and navigating to C:/Program Files (x86)/Xpra, all the files look intact. Also, the installer will ask to close Xpra before letting you install it.

The uninstaller should do a check to make sure Xpra has been closed.



Tue, 14 Jan 2014 01:54:11 GMT - Antoine Martin: keywords, owner, status, component changed; milestone set

This will require an NSIS plugin and checking the process name (just checking the main xpra_cmd.exe or xpra.exe, not all the other tools): NSIS Check whether your application is running using the name of the process, as we can't rely on the other options.

The same check should apply to the installer (upgrade) and uninstaller.


Wed, 05 Feb 2014 08:37:28 GMT - Antoine Martin: owner, status changed

The link in comment:1 would have been useful if we had been using NSIS... but since we're using InnoSetup instead, here is a solution based on:

We only check for 3 process names: Xpra.exe, Xpra_cmd.exe and Xpra-Launcher.exe. So if the users somehow finds his way to one of the other EXE files (ie: Encoding_info.exe, GStreamer_info.exe...) they will still get the error/warning, we will have to assume they are clever enough to figure it out from there.


Tue, 18 Mar 2014 23:34:45 GMT - Smo:

I can help do this. Just not sure where to place psvince.dll.


Wed, 19 Mar 2014 01:16:46 GMT - Antoine Martin:

Oops, it's already done, the changeset for this is r5362.


Thu, 20 Mar 2014 17:41:59 GMT - J. Max Mena:

Retested with r5828, uninstaller shows a pop-up that tells the user they must stop Xpra before uninstalling.


Fri, 21 Mar 2014 00:37:21 GMT - Antoine Martin: status changed; resolution set

Done!

PS: caused #560


Sat, 23 Jan 2021 04:57:20 GMT - migration script:

this ticket has been moved to: https://github.com/Xpra-org/xpra/issues/495