[PATCH] bugfix to 'bzr ignore'

David Allouche david at allouche.net
Sat May 14 16:10:42 BST 2005


Hello folks,

tried to play around with bzr a bit, found a bug, fixed the bug. Here's
the patch.

The problem was that command_ignore did "b.controlfile(ifn, 'rt')" and
'rt' was not longer accept as a legal mode for control files.

Since it appears that the policy is that all control files must be used
in binary more to dodge newline handling insanity, I modified
command_ignore to be consistent.

I also added blackblox test suite coverage for 'bzr ignore' when
the .bzrignore file already exists.

Actually, I got there trying to build the API documentation. So here is
another patch that makes ./buildapi actually work on fresh checkout.

Cheers.
-- 
                                                            -- ddaa
-------------- next part --------------
A non-text attachment was scrubbed...
Name: bzr-ignore.patch
Type: text/x-patch
Size: 1447 bytes
Desc: 
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20050514/9902acc4/attachment.bin 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: bzr-buildapi.patch
Type: text/x-patch
Size: 302 bytes
Desc: 
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20050514/9902acc4/attachment-0001.bin 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20050514/9902acc4/attachment.pgp 


More information about the bazaar mailing list