Merge lp:~michihenning/apparmor-easyprof-ubuntu/new-thumbnailer-methods into lp:apparmor-easyprof-ubuntu
Status: | Merged | ||||
---|---|---|---|---|---|
Merged at revision: | 57 | ||||
Proposed branch: | lp:~michihenning/apparmor-easyprof-ubuntu/new-thumbnailer-methods | ||||
Merge into: | lp:apparmor-easyprof-ubuntu | ||||
Diff against target: |
66 lines (+11/-4) 5 files modified
data/policygroups/ubuntu/1.0/audio (+1/-1) data/templates/ubuntu/1.0/ubuntu-sdk (+1/-1) data/templates/ubuntu/1.1/ubuntu-sdk (+1/-1) data/templates/ubuntu/1.3/ubuntu-sdk (+1/-1) debian/changelog (+7/-0) |
||||
To merge this branch: | bzr merge lp:~michihenning/apparmor-easyprof-ubuntu/new-thumbnailer-methods | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Jamie Strandboge (community) | Needs Information | ||
Pat McGowan | Pending | ||
James Henstridge | Pending | ||
Review via email: mp+281712@code.launchpad.net |
Commit message
Added TraceClient and MaxBacklog dbus methods to the list of allowed methods for applications using the thumbnailer.
Description of the change
Added ClientConfig dbus method to the list of allowed methods for applications using the thumbnailer. The method returns configuration settings that are required by our client API, namely, whether to write thumbnailer-related trace messages into the application log, and how many outstanding dbus requests will be sent by the client before they go into a queue.
Because confined apps cannot access gsettings, and because trying to control this via environment variables is really awkward, the easiest option was to add a dbus method to provide these values to the client. The method returns gsettings values we need on the client side (see man thumbnailer-
Related MR: https:/
Please hold this review for the moment. We'll definitely need TraceClient, but we may not need MaxBacklog.