Merge lp:~osomon/webbrowser-app/aa-profile-allow-dev-shm into lp:webbrowser-app

Proposed by Olivier Tilloy on 2015-10-21
Status: Merged
Approved by: Olivier Tilloy on 2015-10-22
Approved revision: 1243
Merged at revision: 1250
Proposed branch: lp:~osomon/webbrowser-app/aa-profile-allow-dev-shm
Merge into: lp:webbrowser-app
Diff against target: 12 lines (+2/-1)
1 file modified
debian/rules (+2/-1)
To merge this branch: bzr merge lp:~osomon/webbrowser-app/aa-profile-allow-dev-shm
Reviewer Review Type Date Requested Status
PS Jenkins bot continuous-integration Approve on 2015-10-21
Ubuntu Phablet Team 2015-10-21 Pending
Review via email: mp+275200@code.launchpad.net

Commit message

Modify the generated apparmor profile to allow rw access to /dev/shm/.org.chromium.Chromium.* too.

To post a comment you must log in.
Jamie Strandboge (jdstrand) wrote :

It is probably easier to read if you do:
sed 's#/run/shm/\.org\.chromium\.Chromium\.\*#/{dev,run}/shm/.org.chromium.Chromium.*#g'

1243. By Olivier Tilloy on 2015-10-21

Use alternative sed syntax to make the substitution more readable, and escape dots.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/rules'
2--- debian/rules 2015-09-29 21:28:51 +0000
3+++ debian/rules 2015-10-21 15:05:47 +0000
4@@ -13,7 +13,8 @@
5 apparmor:
6 aa-easyprof -m ./debian/webbrowser-app-apparmor.manifest --no-verify | \
7 egrep -v '(# Click packages|CLICK_DIR)' | \
8- sed 's/@{APP_PKGNAME}_@{APP_APPNAME}_@{APP_VERSION}/@{APP_PKGNAME}/g' \
9+ sed 's/@{APP_PKGNAME}_@{APP_APPNAME}_@{APP_VERSION}/@{APP_PKGNAME}/g' | \
10+ sed 's#/run/shm/\.org\.chromium\.Chromium\.\*#/{dev,run}/shm/.org.chromium.Chromium.*#g' \
11 > ./debian/usr.bin.webbrowser-app
12 apparmor_parser -QTK ./debian/usr.bin.webbrowser-app
13

Subscribers

People subscribed via source and target branches

to status/vote changes: