Merge lp:~cypressyew/checkbox/touchpad_driver_info_fix into lp:checkbox

Proposed by Po-Hsu Lin on 2014-08-19
Status: Merged
Approved by: Daniel Manrique on 2014-08-19
Approved revision: 3202
Merged at revision: 3202
Proposed branch: lp:~cypressyew/checkbox/touchpad_driver_info_fix
Merge into: lp:checkbox
Diff against target: 11 lines (+1/-1)
1 file modified
providers/plainbox-provider-checkbox/bin/touchpad_driver_info (+1/-1)
To merge this branch: bzr merge lp:~cypressyew/checkbox/touchpad_driver_info_fix
Reviewer Review Type Date Requested Status
Daniel Manrique 2014-08-19 Approve on 2014-08-19
Review via email: mp+231295@code.launchpad.net

Description of the change

Small fix that makes the device category consistent with the category we set in the job file

To post a comment you must log in.
Daniel Manrique (roadmr) wrote :

Perfect, thanks!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'providers/plainbox-provider-checkbox/bin/touchpad_driver_info'
2--- providers/plainbox-provider-checkbox/bin/touchpad_driver_info 2014-01-07 13:43:22 +0000
3+++ providers/plainbox-provider-checkbox/bin/touchpad_driver_info 2014-08-19 04:18:20 +0000
4@@ -15,7 +15,7 @@
5 attributes = {}
6
7 def addDevice(self, device):
8- if getattr(device, 'category') == 'TOUCH':
9+ if getattr(device, 'category') == 'TOUCHPAD':
10 self.attributes['driver'] = getattr(device, 'driver')
11 self.attributes['product'] = getattr(device, 'product')
12

Subscribers

People subscribed via source and target branches