[MERGE] bzr rm should delete the working file (Bug #82602)
Marius Kruger
amanic at gmail.com
Fri Apr 20 06:15:11 BST 2007
hi,
On 4/19/07, John Arbash Meinel <john at arbash-meinel.com> wrote:
>
> The only part I don't like is the "magic" for taking
>
> self.run_bzr('command to run')
>
> There are a few issues... The biggest being that shlex.split() doesn't
> support Unicode arguments.
all the tests pass, and if someone have a unique test and we cant fix it
by encoding/decoding (like we did in command .get_alias ) they can
just fall back to self.run_bzr('command', 'to', 'run')
I'm not sure if the others caught that you changed this, but it is
> something that we haven't done (in favor of being very explicit about
> what arguments are being passed by run_bzr).
Obviously, I don't think its that a bad an idea.
I would rather something like this wasn't "snuck-in" without discussion
> on a merge for "bzr rm".
>
(I'm not strictly opposed to the change, though I'm a little negative on
> it. I am opposed to it being merged without discussion).
agreed :)
The reason I made it like this is because runbzr did it, and I found
simplified my tests, but you didn't like me using it.
If you would like I can move this magic to my TestCase
and I can resubmit moving it into run_bzr later for more descussion.
--
I don't trust a Revision Control System with less than 5900 unit tests.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: https://lists.ubuntu.com/archives/bazaar/attachments/20070420/13722dd6/attachment.htm
More information about the bazaar
mailing list