Merge lp:~zyga/checkbox/fix-1092925 into lp:checkbox

Proposed by Zygmunt Krynicki
Status: Merged
Approved by: Daniel Manrique
Approved revision: 3082
Merged at revision: 3082
Proposed branch: lp:~zyga/checkbox/fix-1092925
Merge into: lp:checkbox
Diff against target: 14 lines (+2/-1)
1 file modified
providers/plainbox-provider-checkbox/bin/rotation_test (+2/-1)
To merge this branch: bzr merge lp:~zyga/checkbox/fix-1092925
Reviewer Review Type Date Requested Status
Daniel Manrique (community) Approve
Review via email: mp+223540@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Daniel Manrique (roadmr) wrote :

Thanks! A tiny additional fix is needed, see inline comment.

review: Needs Fixing
Revision history for this message
Zygmunt Krynicki (zyga) wrote :

(empty message aka see the silly inline comment thing)

Revision history for this message
Daniel Manrique (roadmr) wrote :

Sorry! No, apparently I'm the one missing stuff :)

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'providers/plainbox-provider-checkbox/bin/rotation_test'
--- providers/plainbox-provider-checkbox/bin/rotation_test 2014-01-07 13:43:22 +0000
+++ providers/plainbox-provider-checkbox/bin/rotation_test 2014-06-18 11:54:00 +0000
@@ -45,8 +45,9 @@
45 for rot in rotations:45 for rot in rotations:
46 try:46 try:
47 status = rotate_screen(rotations[rot])47 status = rotate_screen(rotations[rot])
48 except(xrandr.RRError, xrandr.UnsupportedRRError) as error:48 except (xrandr.RRError, xrandr.UnsupportedRRError) as exc:
49 status = 149 status = 1
50 error = exc
50 else:51 else:
51 error = 'N/A'52 error = 'N/A'
52 # Collect the status and the error message53 # Collect the status and the error message

Subscribers

People subscribed via source and target branches