AppArmor

Merge lp://qastaging/~cameronnemo/apparmor/abstraction-fdo-applications-fixups into lp://qastaging/apparmor/2.12

  1. abstraction-fdo-applications-fixups
  2. Merge into master
Proposed by Cameron Norman
Status: Merged
Merged at revision: 3706
Proposed branch: lp://qastaging/~cameronnemo/apparmor/abstraction-fdo-applications-fixups
Merge into: lp://qastaging/apparmor/2.12
Diff against target: 18 lines (+4/-4)
1 file modified
profiles/apparmor.d/abstractions/freedesktop.org (+4/-4)
To merge this branch: bzr merge lp://qastaging/~cameronnemo/apparmor/abstraction-fdo-applications-fixups
Reviewer Review Type Date Requested Status
Christian Boltz Approve
intrigeri Approve
Review via email: mp+261336@code.qastaging.launchpad.net
To post a comment you must log in.
Revision history for this message
Christian Boltz (cboltz) wrote :

What about merging the rules for main dir and sub dir? You'll get something like

/usr/{,local/}share/applications/{*/,} r,
/usr/{,local/}share/applications/{*/,}defaults.list r,
                                 ^^^^^

Revision history for this message
intrigeri (intrigeri) wrote :

Hi Cameron! Christian's suggestion makes sense to me. Are you still interested in pushing this MR forward, or should someone else take over?

review: Needs Fixing
lp://qastaging/~cameronnemo/apparmor/abstraction-fdo-applications-fixups updated
3112. By Cameron Norman

Merged two rule groups

Revision history for this message
Cameron Norman (cameronnemo) wrote :

Made adjustments suggested

Revision history for this message
intrigeri (intrigeri) wrote :

Looks good to me, I would merge this. Thanks for the update!

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

> Looks good to me, I would merge this. Thanks for the update!

Two months later: bump :)

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

Merged into trunk, 2.11, 2.10 and 2.9 branch.

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

Christian Boltz:
> Merged into trunk, 2.11, 2.10 and 2.9 branch.

Thanks a lot! :)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches