Merge lp:~sylvain-pineau/checkbox/fix-1473941 into lp:checkbox

Proposed by Sylvain Pineau
Status: Merged
Approved by: Sylvain Pineau
Approved revision: 4471
Merged at revision: 4471
Proposed branch: lp:~sylvain-pineau/checkbox/fix-1473941
Merge into: lp:checkbox
Diff against target: 23 lines (+6/-0)
1 file modified
plainbox/plainbox/impl/ctrl.py (+6/-0)
To merge this branch: bzr merge lp:~sylvain-pineau/checkbox/fix-1473941
Reviewer Review Type Date Requested Status
Sylvain Pineau (community) Approve
Review via email: mp+303099@code.launchpad.net

Description of the change

Fixes the linked bug

To post a comment you must log in.
Revision history for this message
Sylvain Pineau (sylvain-pineau) wrote :

self-approved. I'll push a post release to pypi to make use of it to build snaps.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'plainbox/plainbox/impl/ctrl.py'
2--- plainbox/plainbox/impl/ctrl.py 2016-06-20 11:31:35 +0000
3+++ plainbox/plainbox/impl/ctrl.py 2016-08-17 08:10:21 +0000
4@@ -1279,6 +1279,9 @@
5 # Doesn't work for windows jobs
6 if 'win32' in job.get_flag_set():
7 return -1
8+ # Doesn't work with snappy
9+ if (os.getenv("SNAP") or os.getenv("SNAP_APP_PATH")):
10+ return -1
11 # Only makes sense with jobs that need to run as another user
12 # Promote this controller only if the trusted launcher is authorized to
13 # run jobs as another user
14@@ -1374,6 +1377,9 @@
15 # Doesn't work for windows jobs
16 if 'win32' in job.get_flag_set():
17 return -1
18+ # Doesn't work with snappy
19+ if (os.getenv("SNAP") or os.getenv("SNAP_APP_PATH")):
20+ return -1
21 if job.user is not None:
22 return 1
23 else:

Subscribers

People subscribed via source and target branches