VMBuilder

Merge lp:~serge-hallyn/vmbuilder/cleanup-iso into lp:vmbuilder

  1. cleanup-iso
  2. Merge into 0.12
Proposed by Serge Hallyn
Status: Merged
Merged at revision: 460
Proposed branch: lp:~serge-hallyn/vmbuilder/cleanup-iso
Merge into: lp:vmbuilder
Diff against target: 20 lines (+2/-1)
1 file modified
VMBuilder/distro.py (+2/-1)
To merge this branch: bzr merge lp:~serge-hallyn/vmbuilder/cleanup-iso
Reviewer Review Type Date Requested Status
Serge Hallyn Approve
Review via email: mp+43528@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Serge Hallyn (serge-hallyn) wrote :

If anyone is interested in actually reviewing this, please let me know.
Otherwise I'll try to push it tomorrow. (I don't want to circumvent
anyone, but also don't want to pester anyone)

Revision history for this message
Serge Hallyn (serge-hallyn) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'VMBuilder/distro.py'
2--- VMBuilder/distro.py 2010-06-15 19:10:04 +0000
3+++ VMBuilder/distro.py 2010-12-13 15:36:56 +0000
4@@ -66,7 +66,7 @@
5 try:
6 call_hooks(self, *args, **kwargs)
7 except Exception:
8- #self.cleanup()
9+ self.cleanup()
10 raise
11
12 class Distro(Context):
13@@ -83,6 +83,7 @@
14 self.call_hooks('bootstrap')
15 self.call_hooks('configure_os')
16 self.call_hooks('post_install')
17+ self.cleanup()
18
19 def has_xen_support(self):
20 """Install the distro into destdir"""

Subscribers

People subscribed via source and target branches