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

Proposed by Marc Tardif
Status: Merged
Merged at revision: 1652
Proposed branch: lp:~cr3/checkbox/1047883
Merge into: lp:checkbox
Diff against target: 30 lines (+3/-1)
2 files modified
debian/changelog (+1/-0)
patches/0.14.2 (+2/-1)
To merge this branch: bzr merge lp:~cr3/checkbox/1047883
Reviewer Review Type Date Requested Status
Daniel Manrique (community) Approve
Review via email: mp+123637@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Daniel Manrique (roadmr) wrote :

Looks good! Merging...

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/changelog'
2--- debian/changelog 2012-09-10 13:35:43 +0000
3+++ debian/changelog 2012-09-10 20:37:30 +0000
4@@ -43,6 +43,7 @@
5 [Marc Tardif]
6 * scripts/touchpad_scroll_resource: Added support for systems without
7 a touchpad (LP #1045066)
8+ * patch/0.14.2: Fixed patch to rmtree instead of rmdir scripts directory.
9
10 [Sean Feole]
11 * [FEATURE] scripts/battery_test: measures battery capacity before and after
12
13=== modified file 'patches/0.14.2'
14--- patches/0.14.2 2012-07-13 19:58:25 +0000
15+++ patches/0.14.2 2012-09-10 20:37:30 +0000
16@@ -2,6 +2,7 @@
17
18 import os
19 import sys
20+import shutil
21
22
23 def main(args):
24@@ -9,7 +10,7 @@
25 share_dir = "/usr/share/checkbox/scripts"
26 lib_dir = "../../lib/checkbox/bin"
27 if os.path.isdir(share_dir) and not os.path.islink(share_dir):
28- os.rmdir(share_dir)
29+ shutil.rmtree(share_dir)
30 os.symlink(lib_dir, share_dir)
31
32

Subscribers

People subscribed via source and target branches