mediascanner2

Merge lp:~alextu/mediascanner2/extend_permission_to_all_media_codecs_xml into lp:mediascanner2

  1. extend_permission_to_all_media_codecs_xml
  2. Merge into trunk
Proposed by Alex Tu
Status: Merged
Approved by: James Henstridge
Approved revision: 328
Merged at revision: 328
Proposed branch: lp:~alextu/mediascanner2/extend_permission_to_all_media_codecs_xml
Merge into: lp:mediascanner2
Diff against target: 12 lines (+1/-1)
1 file modified
debian/usr.lib.mediascanner-2.0.mediascanner-extractor (+1/-1)
To merge this branch: bzr merge lp:~alextu/mediascanner2/extend_permission_to_all_media_codecs_xml
Reviewer Review Type Date Requested Status
Alfonso Sanchez-Beato (community) Approve
James Henstridge Approve
Jim Hodapp (community) code Approve
You-Sheng Yang Pending
Review via email: mp+293636@code.launchpad.net

Commit message

apparmor: extend read permissions to cover media_codecs*.xml to support Turbo.

Description of the change

apparmor: extend read permissions to cover media_codecs*.xml to support Turbo.

To post a comment you must log in.
Revision history for this message
Jim Hodapp (jhodapp) wrote :

This is consistent with media-hub's apparmor rule, so looks good.

review: Approve (code)
Revision history for this message
James Henstridge (jamesh) wrote :

From the comments from Jamie in the bug report, this looks fine.

review: Approve
Revision history for this message
Alfonso Sanchez-Beato (alfonsosanchezbeato) wrote :

LGTM

review: Approve
Revision history for this message
Alex Tu (alextu) wrote :

silo:
https://launchpad.net/~ci-train-ppa-service/+archive/ubuntu/landing-079/+packages

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/usr.lib.mediascanner-2.0.mediascanner-extractor'
2--- debian/usr.lib.mediascanner-2.0.mediascanner-extractor 2015-11-03 11:27:57 +0000
3+++ debian/usr.lib.mediascanner-2.0.mediascanner-extractor 2016-05-03 15:07:14 +0000
4@@ -90,7 +90,7 @@
5 deny /{,var/}run/user/*/orcexec* w,
6 deny @{HOME}/orcexec* w,
7
8- /{,android/}system/etc/media_codecs.xml r,
9+ /{,android/}system/etc/media_codecs*.xml r,
10 /etc/wildmidi/wildmidi.cfg r,
11
12 # Allow read on all directories

Subscribers

People subscribed via source and target branches

to all changes: