Merge lp:~lifeless/bzr-builddeb/trunk into lp:bzr-builddeb

Proposed by Robert Collins
Status: Rejected
Rejected by: Robert Collins
Proposed branch: lp:~lifeless/bzr-builddeb/trunk
Merge into: lp:bzr-builddeb
Diff against target: 18 lines (+1/-1)
1 file modified
changes.py (+1/-1)
To merge this branch: bzr merge lp:~lifeless/bzr-builddeb/trunk
Reviewer Review Type Date Requested Status
James Westby Disapprove
Review via email: mp+23749@code.launchpad.net

Description of the change

This fixes a relative import which causes breaks testing looms.

To post a comment you must log in.
Revision history for this message
James Westby (james-w) wrote :

Hi,

As discussed before this isn't appropriate.

There is no

  bzrlib.plugins.builddeb.commands

module, so this won't work.

It's the stdlib commands module that is being imported.

What does this break for you?

Thanks,

James

review: Disapprove
Revision history for this message
Robert Collins (lifeless) wrote :

Mea culpa, I forgot that that is a global module.

Unmerged revisions

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'changes.py'
--- changes.py 2009-12-01 17:14:15 +0000
+++ changes.py 2010-04-20 07:44:16 +0000
@@ -18,13 +18,13 @@
18# Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA18# Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
19#19#
2020
21import commands
22import os21import os
2322
24from debian_bundle import deb82223from debian_bundle import deb822
2524
26from bzrlib.trace import mutter25from bzrlib.trace import mutter
2726
27from bzrlib.plugins.builddeb import commands
28from bzrlib.plugins.builddeb.errors import DebianError, MissingChanges28from bzrlib.plugins.builddeb.errors import DebianError, MissingChanges
2929
30class DebianChanges(deb822.Changes):30class DebianChanges(deb822.Changes):

Subscribers

People subscribed via source and target branches