Skip to content
This repository has been archived by the owner on Feb 13, 2024. It is now read-only.

Not removing all software #13

Closed
shadownetdev1 opened this issue Apr 12, 2017 · 5 comments
Closed

Not removing all software #13

shadownetdev1 opened this issue Apr 12, 2017 · 5 comments

Comments

@shadownetdev1
Copy link

It is my understanding that an uninstaller is suppose to remove its software, so shouldn't ppa-purge be removed after it does it's job on lines 92 & 93?

@morphis
Copy link
Member

morphis commented Apr 14, 2017

@shadownetdev1 this is tricky. We would need to record if ppa-purge was already installed before the installer was executed the first time as otherwise we do something the user doesn't expect from us as he installed ppa-purge on his own rather than we did it. Another option would be that we ship ppa-purge in the snap and don't require its installation at all.

@shadownetdev1
Copy link
Author

shadownetdev1 commented Apr 14, 2017

@morphis My suggestion would be to check for an output from which. It is installed on all Ubuntu distros and to the best of my knowledge most Linux distros

It should be something like this:

   if [ -e /etc/apt/sources.list.d/morphis-ubuntu-anbox-support-xenial.list ]; then
           if command -v ppa-purge >/dev/null 2>&1; then
                 sudo ppa-purge ppa:morphis/anbox-support
           else
                 sudo apt install -y ppa-purge
                 sudo ppa-purge ppa:morphis/anbox-support
                 sudo apt remove -y ppa-purge
           fi
   fi

@shadownetdev1
Copy link
Author

@morphis Updated my comment

@morphis
Copy link
Member

morphis commented Apr 26, 2017

@shadownetdev1 Can you propose this as PR?

@morphis
Copy link
Member

morphis commented May 4, 2017

This is now solved in master.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants