Merge lp:~cr3/checkbox/925582 into lp:checkbox

Proposed by Marc Tardif
Status: Merged
Merged at revision: 1220
Proposed branch: lp:~cr3/checkbox/925582
Merge into: lp:checkbox
Diff against target: 33 lines (+12/-1)
2 files modified
checkbox/parsers/udevadm.py (+5/-1)
debian/changelog (+7/-0)
To merge this branch: bzr merge lp:~cr3/checkbox/925582
Reviewer Review Type Date Requested Status
Daniel Manrique (community) Approve
Review via email: mp+91352@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Daniel Manrique (roadmr) wrote :

Thanks!

Merging...

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'checkbox/parsers/udevadm.py'
2--- checkbox/parsers/udevadm.py 2011-10-11 20:15:01 +0000
3+++ checkbox/parsers/udevadm.py 2012-02-02 21:23:18 +0000
4@@ -270,7 +270,11 @@
5
6 @property
7 def path(self):
8- return self._environment.get("DEVPATH")
9+ devpath = self._environment.get("DEVPATH")
10+ if self._environment.get("DEVTYPE") == "disk" and self._stack:
11+ devpath = re.sub(r"/[^/]+/[^/]+$", "", devpath)
12+
13+ return devpath
14
15 @property
16 def product_id(self):
17
18=== modified file 'debian/changelog'
19--- debian/changelog 2012-02-01 21:12:28 +0000
20+++ debian/changelog 2012-02-02 21:23:18 +0000
21@@ -1,5 +1,12 @@
22 checkbox (0.13.1) precise; urgency=low
23
24+ [Marc Tardif]
25+ * Removed trailing directories from the devpath of disk devices (LP: #925582)
26+
27+ -- Marc Tardif <marc@ubuntu.com> Thu, 02 Feb 2012 16:17:41 -0500
28+
29+checkbox (0.13.1) precise; urgency=low
30+
31 [Brendan Donegan]
32 * Fixed the cpu_topology script so that it doesn't mistake the word
33 'processor' in the value of another field for the field 'processor'

Subscribers

People subscribed via source and target branches