Code review comment for lp:~intrigeri/apparmor-profiles/pidgin-vs-gstreamer-1.6

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

« Back to merge proposal