Merge lp://qastaging/~chadadavis/checkbox/788514 into lp://qastaging/checkbox

Proposed by Chad A. Davis
Status: Merged
Merged at revision: 939
Proposed branch: lp://qastaging/~chadadavis/checkbox/788514
Merge into: lp://qastaging/checkbox
Diff against target: 157 lines (+31/-33)
5 files modified
debian/changelog (+6/-0)
debian/compat (+1/-1)
debian/control (+2/-6)
debian/rules (+18/-23)
po/checkbox.pot (+4/-3)
To merge this branch: bzr merge lp://qastaging/~chadadavis/checkbox/788514
Reviewer Review Type Date Requested Status
Marc Tardif (community) Approve
Barry Warsaw (community) Approve
Review via email: mp+65742@code.qastaging.launchpad.net

Description of the change

Transition from pycentral+cdbs to dh_python2/debhelper7
Tested install and basic run in oneiric live. Except for checkbox-urwid. Doesn't seem that python-urwid is on the live CD.

To post a comment you must log in.
938. By Chad A. Davis

Removed README from debian/*.install

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

I helped Chad with this on #ubuntu-pyjam as part of the dhpy2 transition jam. It works great for me, and I approve the patch. I'll happily sponsor an upload after it's merged into trunk.

Chad, thanks so much for your contribution to Ubuntu!

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

Bumping debian/compat to 7 eliminates the need for the debian/tmp prefixes in the .install files:

http://pastebin.ubuntu.com/631827/

Revision history for this message
Marc Tardif (cr3) wrote :

Chad, could you please merge Barry's pastebined diff and I'll have another look at this merge request which should become shorter. Thanks for the awesome work folks!

939. By Chad A. Davis

barrys patch for debhelper7

Revision history for this message
Chad A. Davis (chadadavis) wrote :

Thanks barry, I was looking for something to allow the install files to remain unchanged. Thanks for finding it.

Revision history for this message
Marc Tardif (cr3) wrote :

After testing and looking at the diff of the resulting packages on oneiric, all good. Many thanks for these changes!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches