Merge lp:~darkxst/ppa-purge/lp706774 into lp:ppa-purge
Proposed by
Tim
on 2012-11-29
| Status: | Merged |
|---|---|
| Merge reported by: | Tim |
| Merged at revision: | not available |
| Proposed branch: | lp:~darkxst/ppa-purge/lp706774 |
| Merge into: | lp:ppa-purge |
| Diff against target: |
67 lines (+18/-6) 3 files modified
debian/changelog (+6/-0) debian/control (+1/-1) ppa-purge (+11/-5) |
| To merge this branch: | bzr merge lp:~darkxst/ppa-purge/lp706774 |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Robert Hooker | 2013-01-22 | Pending | |
| ppa-purge | 2012-11-29 | Pending | |
|
Review via email:
|
|||
Description of the Change
Specify removal of packages (via $REINSTALL) that don't exist in the ubuntu archives. This allows apt-get to handle the ppa purge in most cases.
I have removed the dependency on aptitude, however leaving the fallback code in the script, since there may be rare cases where apt-get will fail if for some reason we are left with missing/unsatisfied dependencies. Although I have not seen this happen in my testing.
To post a comment you must log in.
lp:~darkxst/ppa-purge/lp706774
updated
on 2012-12-03
| Tim (darkxst) wrote : | # |

Would be good to get this merged. I have noticed some cases where aptitude still fails to work 100% correctly, when running without this patch.