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

Make Windows uninstaller to remove only installed files and not all files in the installation directory #606

Closed
Raildex opened this issue Mar 31, 2023 · 2 comments · Fixed by #636

Comments

@Raildex
Copy link

Raildex commented Mar 31, 2023

I installed the Media Writer into a wrong directory.
(one directory up where it was supposed to be).
I used the deinstaller and it fucking deleted all content of my parent folder!

@simmon-nplob
Copy link
Contributor

simmon-nplob commented Mar 31, 2023

Interesting. That's not possible, how could it be?
Unless there is a special reason, you should use dnf to install and use it.
I'll have to test it to see if it's the same as your case.

  • test - Fedora37 server-kde/install mediawriter (remove mediawriter after specific foler install)

      rpm -iq mediawriter-5.0.3-1.fc37.src.rpm 
    

it does not effect any remove folder or file~

where are you install ???

@grulja
Copy link
Collaborator

grulja commented Mar 31, 2023

I installed the Media Writer into a wrong directory. (one directory up where it was supposed to be). I used the deinstaller and it fucking deleted all content of my parent folder!

That's really unfortunate and I'm sorry. I can see the uninstaller removes everything within the same folder. We will need to make it to uninstall only the files we installed.

@grulja grulja added the Windows label Mar 31, 2023
@grulja grulja changed the title Deinstaller deinstalls all subfolders! Make Windows uninstaller to remove only installed files and not all files in the installation directory Mar 31, 2023
@grulja grulja added the bug label Mar 31, 2023
grulja added a commit to grulja/MediaWriter that referenced this issue Sep 1, 2023
- only uninstall files we install (fixes FedoraQt#606)
- add some additional missing properties (fixes FedoraQt#572)
grulja added a commit to grulja/MediaWriter that referenced this issue Sep 1, 2023
- only uninstall files we install (fixes FedoraQt#606)
- add some additional missing properties (fixes FedoraQt#572)
grulja added a commit to grulja/MediaWriter that referenced this issue Sep 1, 2023
- only uninstall files we install (fixes FedoraQt#606)
- add some additional missing properties (fixes FedoraQt#572)
grulja added a commit to grulja/MediaWriter that referenced this issue Sep 1, 2023
- only uninstall files we install (fixes FedoraQt#606)
- add some additional missing properties (fixes FedoraQt#572)
grulja added a commit to grulja/MediaWriter that referenced this issue Sep 1, 2023
- only uninstall files we install (fixes FedoraQt#606)
- add some additional missing properties (fixes FedoraQt#572)
grulja added a commit to grulja/MediaWriter that referenced this issue Sep 1, 2023
- only uninstall files we install (fixes FedoraQt#606)
- add some additional missing properties (fixes FedoraQt#572)
grulja added a commit to grulja/MediaWriter that referenced this issue Sep 1, 2023
- only uninstall files we install (fixes FedoraQt#606)
- add some additional missing properties (fixes FedoraQt#572)
grulja added a commit that referenced this issue Sep 1, 2023
- only uninstall files we install (fixes #606)
- add some additional missing properties (fixes #572)
grulja added a commit that referenced this issue Sep 4, 2023
- only uninstall files we install (fixes #606)
- add some additional missing properties (fixes #572)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants