Merge lp:~osomon/evince/snap-allow-printing into lp:~ubuntu-desktop/evince/snap

Proposed by Olivier Tilloy
Status: Merged
Merged at revision: 12
Proposed branch: lp:~osomon/evince/snap-allow-printing
Merge into: lp:~ubuntu-desktop/evince/snap
Diff against target: 21 lines (+4/-0)
1 file modified
snapcraft.yaml (+4/-0)
To merge this branch: bzr merge lp:~osomon/evince/snap-allow-printing
Reviewer Review Type Date Requested Status
Ken VanDine Approve
Review via email: mp+343521@code.launchpad.net

Commit message

Add cups-control plug to allow printing.

To post a comment you must log in.
Revision history for this message
Ken VanDine (ken-vandine) wrote :

Good catch, I think we need avahi-observe as well.

13. By Olivier Tilloy

Add avahi-observe plug too.

Revision history for this message
Ken VanDine (ken-vandine) wrote :

Thanks!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'snapcraft.yaml'
2--- snapcraft.yaml 2018-03-20 16:16:33 +0000
3+++ snapcraft.yaml 2018-04-18 14:58:58 +0000
4@@ -25,6 +25,8 @@
5 evince:
6 command: desktop-launch evince
7 plugs:
8+ - avahi-observe
9+ - cups-control
10 - desktop
11 - desktop-legacy
12 - gsettings
13@@ -36,6 +38,8 @@
14 evince-previewer:
15 command: desktop-launch evince-previewer
16 plugs:
17+ - avahi-observe
18+ - cups-control
19 - desktop
20 - desktop-legacy
21 - gsettings

Subscribers

People subscribed via source and target branches

to all changes: