Merge lp:~maaarc-deactivatedaccount-deactivatedaccount-deactivatedaccount-deactivatedaccount-deactivatedaccount/ubuntu/natty/checkbox/checkbox-fix-0.10.4 into lp:ubuntu/natty/checkbox

Proposed by Barry Warsaw
Status: Merged
Merge reported by: Barry Warsaw
Merged at revision: not available
Proposed branch: lp:~maaarc-deactivatedaccount-deactivatedaccount-deactivatedaccount-deactivatedaccount-deactivatedaccount/ubuntu/natty/checkbox/checkbox-fix-0.10.4
Merge into: lp:ubuntu/natty/checkbox
Diff against target: 34 lines (+7/-2)
2 files modified
checkbox/lib/config.py (+1/-2)
debian/changelog (+6/-0)
To merge this branch: bzr merge lp:~maaarc-deactivatedaccount-deactivatedaccount-deactivatedaccount-deactivatedaccount-deactivatedaccount/ubuntu/natty/checkbox/checkbox-fix-0.10.4
Reviewer Review Type Date Requested Status
Barry Warsaw (community) Approve
Ubuntu branches Pending
Review via email: mp+45436@code.launchpad.net

Description of the change

Your change applying Marc's upstream patch improves the fix.

To post a comment you must log in.
Revision history for this message
Barry Warsaw (barry) :
review: Approve
Revision history for this message
Micah Gersten (micahg) wrote :

Shouldn't the version on this be 0.10.4-0ubuntu1 since there's an upstream release tarball?

Revision history for this message
Barry Warsaw (barry) wrote :

Where's the upstream tarball? It's not at:

https://launchpad.net/checkbox

AFAICT.

Revision history for this message
Micah Gersten (micahg) wrote :

There's a watch file pointing to it.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'checkbox/lib/config.py'
2--- checkbox/lib/config.py 2010-12-15 13:05:20 +0000
3+++ checkbox/lib/config.py 2011-01-06 19:57:07 +0000
4@@ -20,7 +20,6 @@
5 import re
6 import logging
7 import posixpath
8-import types
9
10 from ConfigParser import ConfigParser
11
12@@ -38,7 +37,7 @@
13
14 def __setitem__(self, key, value):
15 if key == "includes":
16- if type(value) == types.ListType:
17+ if isinstance(value, list):
18 value = value[0]
19 for path in split(value):
20 path = self._parser._interpolate("DEFAULT", None, path, self)
21
22=== modified file 'debian/changelog'
23--- debian/changelog 2010-12-15 13:05:20 +0000
24+++ debian/changelog 2011-01-06 19:57:07 +0000
25@@ -1,3 +1,9 @@
26+checkbox (0.10.4) natty; urgency=low
27+
28+ * Added Marc Tardif's Fixed parsing of config parameters (LP: #689140)
29+
30+ -- Marc Legris <marc.legris@canonical.com> Thu, 06 Jan 2011 09:51:37 -0500
31+
32 checkbox (0.10.3ubuntu1) natty; urgency=low
33
34 * Hack to work around internals of ConfigParser in python2.7. LP: #689140.

Subscribers

People subscribed via source and target branches

to all changes: