Capomastro

Merge lp://staging/~caio1982/capomastro/debuninstall-1450608 into lp://staging/capomastro

  1. debuninstall-1450608
  2. Merge into trunk
Proposed by Caio Begotti
Status: Merged
Approved by: Daniel Manrique
Approved revision: 201
Merged at revision: 202
Proposed branch: lp://staging/~caio1982/capomastro/debuninstall-1450608
Merge into: lp://staging/capomastro
Diff against target: 1 lines (+0/-0)
0 files modified
To merge this branch: bzr merge lp://staging/~caio1982/capomastro/debuninstall-1450608
Reviewer Review Type Date Requested Status
Daniel Manrique (community) Approve
Review via email: mp+263719@code.staging.launchpad.net

Commit message

Make it possible to cleanly remove the package (useful for development and testing) again

Description of the change

Such a dumb bug with a even dumber fix :-)

We were trying to disable and remove stuff after removing the package, of course it wouldn't work.

To post a comment you must log in.
lp://staging/~caio1982/capomastro/debuninstall-1450608 updated
201. By Caio Begotti

rename the removal script so it executes before the package and its files are actually removed, otherwise of course the rm and a2dissite will fail

Revision history for this message
Daniel Manrique (roadmr) wrote :

:D

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches