[merge] global (per user) ignores and other cleanups

Martin Pool mbp at canonical.com
Mon Jul 10 03:15:11 BST 2006


On  7 Jul 2006, John Arbash Meinel <john at arbash-meinel.com> wrote:
> Attached is a bundle which implements a global ignore file in
> ~/.bazaar/ignore

Just two thing: please add a NEWS entry so people can see
what changed, and please add at least a stub to the documentation
describing that the file exists.

> Anyway, the attached bundle cleans up all of that. I'm not 100%
> convinced that we want a global user ignore, because it is too easy to
> use that, rather than properly setting project ignores.
> For example, I would be very tempted to add *.py[co] to my personal
> global ignores. But that means if someone else checks out my project,
> they will get a bunch of unknown files in their tree.
> 
> I know Wayne Davison feels very strongly that editor temporary files
> should not be in per-project settings. I'm just more concerned about the
> opposite. Where a project should be self-contained to ignore the right
> things. And *.o and *.py[co] are very likely to end up in a user's
> default ignores because they get tired of adding it all the time.

There is a risk that people will confuse themselves, but I think it's
still useful to have them to accomodate personal practices.

> The other possibility would be that if ~/.bazaar/ignore *doesn't* exist,
> it uses the builtin list of default ignores. Which is more of a
> compatible upgrade path than our current system.

Well, I think we would want that list to be more conservative than what
we have at the moment, both to avoid surprising people and to help with
performance.

So I'll give a +1, the patch looks good as written.  If other people
have concerns about adding the feature we can talk about it some more
but I think it's basically reasonable.
-- 
Martin




More information about the bazaar mailing list