Merge ~alextu/plainbox-provider-checkbox:pm-test-fix-typo into plainbox-provider-checkbox:master

Proposed by Alex Tu
Status: Merged
Approved by: Sylvain Pineau
Approved revision: 511ad59ae0855e8be8c39175ebd9211d82724cc6
Merged at revision: 75a18a2259ad52bddbf9857aca93d69e0a1b815c
Proposed branch: ~alextu/plainbox-provider-checkbox:pm-test-fix-typo
Merge into: plainbox-provider-checkbox:master
Diff against target: 13 lines (+1/-1)
1 file modified
bin/pm_test (+1/-1)
Reviewer Review Type Date Requested Status
Sylvain Pineau (community) Approve
Checkbox Developers Pending
Review via email: mp+382114@code.launchpad.net

Description of the change

it should be a typo from 45b4ffa9 which break functionality

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

Good catch, thanks Alex

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/bin/pm_test b/bin/pm_test
2index 3dcdf28..066f757 100755
3--- a/bin/pm_test
4+++ b/bin/pm_test
5@@ -39,7 +39,7 @@ def main():
6 username = os.getenv('NORMAL_USER')
7 if not username:
8 uid = os.getenv('SUDO_UID') or os.getenv('PKEXEC_UID')
9- if not uid:q
10+ if not uid:
11 sys.stderr.write('Unable to determine invoking user\n')
12 sys.exit(1)
13 username = pwd.getpwuid(int(uid)).pw_name

Subscribers

People subscribed via source and target branches