[patch] fix bug 30190 "'bzr cat' requires full path in old revision"

John Arbash Meinel john at arbash-meinel.com
Tue Oct 31 20:54:14 GMT 2006


Cheuksan Edward Wang wrote:
> Sorry about that. I have fixed the test. Somehow my indentation bug can
> get past selftest.
> 
> On 10/16/06, *Martin Pool* < mbp at canonical.com
> <mailto:mbp at canonical.com>> wrote:
> 
>     On 16 Oct 2006, Cheuksan Edward Wang < wang02139 at gmail.com
>     <mailto:wang02139 at gmail.com>> wrote:
>     > Thanks. This new changeset should be doing what Robert suggested.
> 
>     There's a syntax error in the testsuite, it won't even run the test.
> 
>     --
>     Martin
> 

Okay, I think it looks good enough to merge. +1 from me, I'll submit a
merge request.

Your indentation made it past selftest because it is just another
function defined in that module. And since it wasn't a member of the a
TestCase derived class, it just looked like a helper function that
shouldn't be run.

John
=:->


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 254 bytes
Desc: OpenPGP digital signature
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20061031/79033ae5/attachment.pgp 


More information about the bazaar mailing list