[MERGE] cleanup blackbox tests

Martin Pool mbp at sourcefrog.net
Wed Jun 27 00:32:15 BST 2007


On 6/27/07, John Arbash Meinel <john at arbash-meinel.com> wrote:
> I don't really like having both a shlex and a varargs. I don't think there is
> any chance for confusion, because we won't ever support having bzr commands
> that take a space in their name. But as a human, it seems a bit ambiguous.
>
> So if you specifically prefer to have shlex available, then it seems reasonable
> to require either "self.run_bzr(['x', 'y', 'z'])" or "self.run_bzr('x y z')".

I don't like it either, the patch was just getting long enough
already.  (You can see that in fact I did deprecate it, then commented
out the deprecation.)  I'm happy to review/merge Vincent's patch to do
that.

-- 
Martin



More information about the bazaar mailing list