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.

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

[Feature request] Uninstaller should remove only files related to the DLSS software #201

Open
jigmcgalliger opened this issue Jan 30, 2024 · 3 comments
Labels
feature request New feature or request

Comments

@jigmcgalliger
Copy link

Is your feature request related to a problem? Please describe.
Uninstalling DLSS swapper deletes all files in the install folder, even if unrelated to the DLSS software. If you have files in the install folder (like game lists) or accidentally install DLSS swapper in a parent directory, there is no way to uninstall it without also removing everything else in that folder.

Describe the solution you'd like
The DLSS swapper software should have context of what files are created by it during the installation and use of the software and only remove those files.

@jigmcgalliger jigmcgalliger added the feature request New feature or request label Jan 30, 2024
@beeradmoore
Copy link
Owner

As of v1.0.4 it does have context aware uninstall. Is this not working as expected or?

@jigmcgalliger
Copy link
Author

jigmcgalliger commented Jan 30, 2024

I was fully up to date (updated 3 weeks ago). It deleted everything within the install folder (just some custom text files with my ideal versions to games, but could be worse if installed in a folder with other files).

Unfortunately, I cannot confirm why it behaved the way it did after the fact.

@beeradmoore
Copy link
Owner

It did exactly that to another user which is why we did multiple updates to both install path validation and uninstall scripts.

They changed the install path to E:\Games\ and they (rightfully) thought it would install to E:\Games\DLSS Swapper. Apparently this isn't how many installers work, us included 💀

So system was updated so if you select a path that doesn't contain the letters "dlss" it will assume you want to append "DLSS Swapper"

On top of that when it installs it builds a list of files in uninstall.dat and so when it uninstalls it will only remove those. If you add other files to your directory the install folder (eg. E:\Games\DLSS Swapper) wont be removed and only the installed files should remain.

Is it possible the updater didnt get you to 1.0.4 so you still had the version with the issues or do you recall installing 1.0.4? (There should be a log in %TEMP% for DLSS Swapper, in that it prints its version on launch.

If you don't think any of these are what happens I'll go do more testing on upgrading and see if I can replicate that behaviour to improve it for the next release.

I'm very sorry you lost data, I 100% want to do all I can to prevent that happening to anyone again because I know I wouldn't enjoy it if it were me

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants