[PATCH] fix for bzr rm -v

Martin Pool mbp at sourcefrog.net
Mon Dec 12 07:24:48 GMT 2005


On  9 Dec 2005, Wouter van Heyst <larstiq at larstiq.dyndns.org> wrote:
> Attached is a testcase and oneline fix for 'bzr rm -v hello.txt' (as
> suggested in http://bazaar.canonical.com/IntroductionToBzr) failing
> because show_status is not known.

Thanks, merged.

> My first guess was to use bzrlib.status.show_status. Wrong idea,
> the code actually wants to use bzrlib.textui.show_status. Now, this does
> make it work again, but I'm not sure textui should be used from
> WorkingTree. What do others think?

As John says, it's just not a good name for it.  Possibly it should be
just removed, because its handling of the kind characters is redundant
with what the inventory entries now do.

-- 
Martin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20051212/63c5acdc/attachment.pgp 


More information about the bazaar mailing list