ignore adds to .bzrignore even if pattern is already ignored

Bug #237438 reported by Russ Brown
4
Affects Status Importance Assigned to Milestone
Bazaar
Fix Released
Medium
Unassigned

Bug Description

Follow these steps:

bzr init test
cd test
touch file
bzr ignore file
bzr ignore file
cat .bzrignore

Note how 'test' is in the file twice.

This is a simple case, but you could also argue that the following shouldn't result in the second item being added either:

bzr init test
cd test
touch file.so
bzr ignore '*.so'
bzr ignore file.so
cat .bzrignore

...since the file is already ignored by the first rule.

Revision history for this message
Martin Albisetti (beuno) wrote :

This adds more cycles to compare files, so we should really plug this.

Changed in bzr:
importance: Undecided → Medium
status: New → Confirmed
Jelmer Vernooij (jelmer)
tags: added: ignore performance
Jelmer Vernooij (jelmer)
tags: added: check-for-breezy
Revision history for this message
Jelmer Vernooij (jelmer) wrote :

This is fixed in modern versions of Bazaar.

tags: removed: check-for-breezy
no longer affects: brz
Changed in bzr:
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.