Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[FR]: Uninstaller should detect if an instance is running #529

Closed
vlakoff opened this issue Nov 1, 2021 · 2 comments
Closed

[FR]: Uninstaller should detect if an instance is running #529

vlakoff opened this issue Nov 1, 2021 · 2 comments
Assignees
@martinrotter
Labels
Status-Fixed Ticket is resolved. Type-Deployment Problem is caused by badly deployed application files. Type-Enhancement This is request for brand new feature.
Milestone
4.3.3

Comments

@vlakoff
Copy link

vlakoff commented Nov 1, 2021 •
edited
Loading

After uninstallation, I noticed the C:\RSS Guard folder was still present, then that there was a running instance.

The uninstaller should detect if an instance is running, and if so, offer to close it programmatically.

Refs in the NSIS documentation: Check whether your application is running
@vlakoff vlakoff added the Type-Enhancement This is request for brand new feature. label Nov 1, 2021
@martinrotter
Copy link
Owner

Indeed.

@martinrotter
Copy link
Owner

fixed, uninstaller now checks if "rssguard.exe" is running and offers to abort uninstallation

@martinrotter martinrotter added this to the 4.3.3 milestone Feb 7, 2023
@martinrotter martinrotter added Status-Fixed Ticket is resolved. Type-Deployment Problem is caused by badly deployed application files. labels Feb 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@martinrotter martinrotter

Labels
Status-Fixed Ticket is resolved. Type-Deployment Problem is caused by badly deployed application files. Type-Enhancement This is request for brand new feature.
Projects
None yet
Milestone
4.3.3
Development

No branches or pull requests
2 participants
@vlakoff @martinrotter