MAAS

Merge lp://staging/~allenap/maas/packaging.move-dhcp-helper-to-lib into lp://staging/~maas-maintainers/maas/packaging

packaging.move-dhcp-helper-to-lib
Merge into packaging

Proposed by Gavin Panella on 2017-03-31

Status:	Rejected
Rejected by:	Gavin Panella on 2017-04-03
Proposed branch:	lp://staging/~allenap/maas/packaging.move-dhcp-helper-to-lib
Merge into:	lp://staging/~maas-maintainers/maas/packaging
Diff against target:	68 lines (+6/-5) 6 files modified debian/changelog (+1/-0) debian/maas-dhcp.apparmor (+1/-1) debian/maas-dhcp.maas-dhcpd.service (+1/-1) debian/maas-dhcp.maas-dhcpd6.service (+1/-1) debian/maas-rack-controller.install (+1/-1) debian/maas-rack-controller.lintian-overrides (+1/-1)
To merge this branch:	bzr merge lp://staging/~allenap/maas/packaging.move-dhcp-helper-to-lib
Related bugs:	Link a bug report

Reviewer	Review Type	Date Requested	Status
Blake Rouse (community)		2017-03-31	Needs Information on 2017-03-31
Review via email: mp+321521@code.staging.launchpad.net

Commit message

Move maas-dhcp-helper off PATH to /usr/lib/maas.

Description of the change

Goes with lp:~allenap/maas/move-dhcp-helper-to-lib.

Revision history for this message

Blake Rouse (blake-rouse) wrote on 2017-03-31:

Looks good.

review: Approve

Revision history for this message

Blake Rouse (blake-rouse) wrote on 2017-03-31:

Again just thought of the reason this might be a problem. Check other branch for reason. A symlink might be needed in this branch and the apparmor conf would need to have both paths.

review: Needs Information

Revision history for this message

Gavin Panella (allenap) wrote on 2017-04-03:

Abandoning this. See https://code.launchpad.net/~allenap/maas/move-dhcp-helper-to-lib/+merge/321520 for discussion.