Inconsistency with directories in "bzr status" vs .bzrignore

Martin Pool mbp at sourcefrog.net
Wed Apr 27 03:26:39 BST 2005


On Wed, 2005-04-27 at 12:23 +1000, Andrew Bennetts wrote:
> On Wed, Apr 27, 2005 at 03:20:12AM +0200, Erik Bågfors wrote:
> [...]
> > *** modified file 'bzrlib/tree.py'
> > --- bzrlib/tree.py
> > +++ bzrlib/tree.py
> > @@ -333,6 +333,8 @@
> >                      newpat = pat[2:]
> >                  else:
> >                      newpat = pat
> > +               if (pat[-1:] == "/") and isdir(filename) and fnmatch.fnmatchcase(filename, newpat[:-1]):
> > +                   return pat
> 
> A nitpick: pat.endswith("/") is a bit clearer than pat[-1:] == "/".

Or indeed pat[-1] == '/'.

I'll merge this but I'd rather fix up/replace fnmatchcase.

-- 
Martin

-------------- 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/20050427/07df04e8/attachment.pgp 


More information about the bazaar mailing list