Merge lp:~andreserl/maas/packaging_updateS_bzr1134 into lp:~maas-maintainers/maas/packaging
Proposed by
Andres Rodriguez
on 2012-10-02
| Status: | Merged |
|---|---|
| Approved by: | Andres Rodriguez on 2012-10-02 |
| Approved revision: | 114 |
| Merged at revision: | 113 |
| Proposed branch: | lp:~andreserl/maas/packaging_updateS_bzr1134 |
| Merge into: | lp:~maas-maintainers/maas/packaging |
| Diff against target: |
181 lines (+38/-33) 6 files modified
debian/changelog (+11/-3) debian/control (+12/-11) debian/maas-cluster-controller.maas-cluster-celery.upstart (+0/-2) debian/maas-dhcp.postinst (+1/-1) debian/maas-region-controller.install (+3/-1) debian/maas-region-controller.postinst (+11/-15) |
| To merge this branch: | bzr merge lp:~andreserl/maas/packaging_updateS_bzr1134 |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Andres Rodriguez | Approve on 2012-10-02 | ||
| Launchpad code reviewers | 2012-10-02 | Pending | |
|
Review via email:
|
|||
Commit Message
* debian/
* debian/
on upgrade. No longer check versioning. (LP: #1060094).
* debian/
* debian/control: Depends on python-netifaces.
* debian/
To post a comment you must log in.
| Scott Moser (smoser) wrote : | # |
lp:~andreserl/maas/packaging_updateS_bzr1134
updated
on 2012-10-02
- 114. By Andres Rodriguez on 2012-10-02
-
wrap-and-sort, comments remove
review:
Approve


probably should wrap-and-sort (at least your additional dependency should be added in such that way).
I'd just remove the code you commented out in postinst.