Merge lp:~morphis/aethercast/fix-apparmor-rules into lp:aethercast

Proposed by Simon Fels
Status: Merged
Approved by: Simon Fels
Approved revision: 186
Merged at revision: 127
Proposed branch: lp:~morphis/aethercast/fix-apparmor-rules
Merge into: lp:aethercast
Diff against target: 52 lines (+18/-9)
3 files modified
debian/control (+1/-0)
debian/rules (+4/-0)
debian/usr.sbin.aethercast (+13/-9)
To merge this branch: bzr merge lp:~morphis/aethercast/fix-apparmor-rules
Reviewer Review Type Date Requested Status
PS Jenkins bot continuous-integration Approve
Thomas Voß (community) Approve
Review via email: mp+287455@code.launchpad.net

Commit message

Fix apparmor rules to let us use the ics-dhcp-server properly.

Description of the change

Fix apparmor rules to let us use the ics-dhcp-server properly.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Simon Fels (morphis) wrote :
Revision history for this message
Thomas Voß (thomas-voss) wrote :

LGTM, feel free to ignore niggles inline.

review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'debian/control'
--- debian/control 2016-01-21 11:13:32 +0000
+++ debian/control 2016-03-02 11:24:40 +0000
@@ -5,6 +5,7 @@
5XSBC-Original-Maintainer: Simon Fels <simon.fels@canonical.com>5XSBC-Original-Maintainer: Simon Fels <simon.fels@canonical.com>
6Build-Depends: cmake,6Build-Depends: cmake,
7 debhelper (>= 9),7 debhelper (>= 9),
8 dh-apparmor,
8 dbus,9 dbus,
9 google-mock,10 google-mock,
10 libboost-dev,11 libboost-dev,
1112
=== modified file 'debian/rules'
--- debian/rules 2016-01-17 14:01:37 +0000
+++ debian/rules 2016-03-02 11:24:40 +0000
@@ -20,3 +20,7 @@
2020
21override_dh_auto_test:21override_dh_auto_test:
22 # disable22 # disable
23
24override_dh_install:
25 dh_install
26 dh_apparmor -paethercast --profile-name=usr.sbin.dhcpd
2327
=== modified file 'debian/usr.sbin.aethercast'
--- debian/usr.sbin.aethercast 2016-01-30 12:33:55 +0000
+++ debian/usr.sbin.aethercast 2016-03-02 11:24:40 +0000
@@ -1,11 +1,15 @@
1# vim:syntax=apparmor1# vim:syntax=apparmor
22
3 # Allow aethercast to start dhcpd with its own configuration3# Work around bug:
4 # file it needs to provide DHCP for groups it manages through4# https://bugs.launchpad.net/ubuntu/+source/isc-dhcp/+bug/1186662
5 # WiFi Direct. This need to stay as long as NetworkManager does5capability dac_override,
6 # not has support for WiFi Direct and aethercast is using that.6
7 /etc/aethercast/dhcpd.conf r,7# Allow aethercast to start dhcpd with its own configuration
8 # In addition aethercast will also point dhcpd to a private8# file. It needs to provide DHCP for groups it manages through
9 # lease/pid file9# WiFi Direct. This needs to stay as long as NetworkManager does
10 /{,var/}run/aethercast/dhcpd*.leases* lrw,10# not have support for WiFi Direct and aethercast is using that.
11 /{,var/}run/aethercast/dhcpd*.pid lrw,11/etc/aethercast/dhcpd.conf r,
12# In addition aethercast will also point dhcpd to a private
13# lease/pid file
14/{,var/}run/aethercast/dhcpd*.leases lrw,
15/{,var/}run/aethercast/dhcpd*.pid lrw,

Subscribers

People subscribed via source and target branches

to all changes: