Merge lp:~intrigeri/apparmor-profiles/pidgin-vs-gstreamer-1.6 into lp:apparmor-profiles

Proposed by intrigeri
Status: Merged
Merged at revision: 154
Proposed branch: lp:~intrigeri/apparmor-profiles/pidgin-vs-gstreamer-1.6
Merge into: lp:apparmor-profiles
Diff against target: 14 lines (+4/-0)
1 file modified
ubuntu/16.04/usr.bin.pidgin (+4/-0)
To merge this branch: bzr merge lp:~intrigeri/apparmor-profiles/pidgin-vs-gstreamer-1.6
Reviewer Review Type Date Requested Status
Christian Boltz Approve
Review via email: mp+277499@code.launchpad.net

Description of the change

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

6 weeks later: ping?

Revision history for this message
Christian Boltz (cboltz) wrote :

I never used pidgin, but the changes look sane and valid, therefore merged into r154.

I tend to think that all gstreamer-related rules should go into abstractions/gstreamer, but
a) that's something for a separate cleanup patch and
b) I don't really know gstreamer, so I'm not sure if all applications using gstreamer also need those permissions (if yes, moving to the abstraction makes sense)

review: Approve
Revision history for this message
intrigeri (intrigeri) wrote :

Christian Boltz wrote (30 Dec 2015 21:03:35 GMT) :
> I tend to think that all gstreamer-related rules should go into abstractions/gstreamer,

Agreed, added to my personal todo list for some day I don't feel like
doing this kind of things.

Cheers,
--
intrigeri

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'ubuntu/16.04/usr.bin.pidgin'
--- ubuntu/16.04/usr.bin.pidgin 2015-10-27 01:49:53 +0000
+++ ubuntu/16.04/usr.bin.pidgin 2015-11-14 13:30:39 +0000
@@ -41,6 +41,10 @@
4141
42 owner @{HOME}/.gstreamer*/ rw,42 owner @{HOME}/.gstreamer*/ rw,
43 owner @{HOME}/.gstreamer*/** rw,43 owner @{HOME}/.gstreamer*/** rw,
44 owner @{HOME}/.cache/gstreamer-[0-9]*.[0-9]*/ rw,
45 owner @{HOME}/.cache/gstreamer-[0-9]*.[0-9]*/registry.*.bin rw,
46 owner @{HOME}/.cache/gstreamer-[0-9]*.[0-9]*/registry.*.bin.tmp* rw,
47 /usr/lib/@{multiarch}/gstreamer[0-9]*.[0-9]*/gstreamer-[0-9]*.[0-9]*/gst-plugin-scanner Pix -> gst_plugin_scanner,
44 owner @{HOME}/.purple/ rw,48 owner @{HOME}/.purple/ rw,
45 owner @{HOME}/.purple/** rwk,49 owner @{HOME}/.purple/** rwk,
46 owner @{HOME}/.config/indicators/ rw,50 owner @{HOME}/.config/indicators/ rw,

Subscribers

People subscribed via source and target branches

to status/vote changes: