saint-13 / Linux_Dynamic_Wallpapers

Dynamic Wallpapers for Linux
742 stars 61 forks source link

Add uninstaller #36

Open benjamin051000 opened 1 year ago

benjamin051000 commented 1 year ago

It's difficult to individually remove each XML file in /usr/share/gnome-background-properties. Please include an ./uninstall.sh that removes these and the wallpapers automatically.

Pbg4444 commented 1 year ago

just remove the whole folder sudo rm -fr /usr/share/gnome-background-properties and sudo rm -fr /usr/share/backgrounds/Dynamic_Wallpapers

benjamin051000 commented 1 year ago

While sudo rm -rf /usr/share/backgrounds/Dynamic_Wallpapers should work, deleting /usr/share/gnome-background-properties appears to have the chance to be destructive to wallpapers not part of this project. On a live image of Fedora 38, you can see that this directory has xml files in it before Linux_Dynamic_Wallpapers is installed: Screenshot from 2023-05-27 08-09-45

Instead, an uninstaller would need to only delete what it had installed and nothing else. It honestly wouldn't be that difficult to accomplish. I also wonder if /usr/share/gnome-background-properties allows for subdirectories, which would make the process even simpler.