default value of grep-find-ignored-directories should include .svn

default value of grep-find-ignored-directories should include .svn

[David Glasser]

The new grep-find-ignored-directories variable is a good idea.  I
think its default value should include ".svn" and probably "_darcs"
too, as these are commonly used version control systems (Subversion at
least is arguably much more popular than Mercurial and Arch, whose
directories are currently in the default value).

--dave
glasser@mit.edu

Re: default value of grep-find-ignored-directories should include .svn

[Kim F. Storm]

David Glasser <glasser@mit.edu> writes:

> The new grep-find-ignored-directories variable is a good idea.  I
> think its default value should include ".svn" and probably "_darcs"
> too, as these are commonly used version control systems (Subversion at
> least is arguably much more popular than Mercurial and Arch, whose
> directories are currently in the default value).

Ok, I have added them.


However, there is generally no need to exclude directories from the
find, unless the version control systems actually store copies/backups
of source files in there, or otherwise use file names which resembles
source file names that may confuse the search and give false matches.

For CVS, in some cases, the CVS/Base subdir contains copies of modified
files.  So CVS must be excluded from the search.

Does the other VC's do this too in some cases?

Or are there other reasons to exclude them from the search?

-- 
Kim F. Storm <storm@cua.dk> http://www.cua.dk

Re: default value of grep-find-ignored-directories should include .svn

[David Glasser]

On 7/19/06, Kim F. Storm <storm@cua.dk> wrote:
> However, there is generally no need to exclude directories from the
> find, unless the version control systems actually store copies/backups
> of source files in there, or otherwise use file names which resembles
> source file names that may confuse the search and give false matches.

Yes, Subversion do store pristine "base" copies of the file (in
.svn/text-base and _darcs/current).

--dave

-- 
David Glasser | glasser@mit.edu | http://www.davidglasser.net/

Re: default value of grep-find-ignored-directories should include .svn

[Kevin Rodgers]

Kim F. Storm wrote:
> David Glasser <glasser@mit.edu> writes:
> 
>> The new grep-find-ignored-directories variable is a good idea.  I
>> think its default value should include ".svn" and probably "_darcs"
>> too, as these are commonly used version control systems (Subversion at
>> least is arguably much more popular than Mercurial and Arch, whose
>> directories are currently in the default value).
> 
> Ok, I have added them.
> 
> However, there is generally no need to exclude directories from the
> find, unless the version control systems actually store copies/backups
> of source files in there, or otherwise use file names which resembles
> source file names that may confuse the search and give false matches.
> 
> For CVS, in some cases, the CVS/Base subdir contains copies of modified
> files.  So CVS must be excluded from the search.
> 
> Does the other VC's do this too in some cases?
> 
> Or are there other reasons to exclude them from the search?

Because your find command may not even have a -name clause, or (as
you say above) the -name pattern may match files in the VC directory.
I think that's a good enough reason.

-- 
Kevin

Re: default value of grep-find-ignored-directories should include .svn

[Stefan Monnier]

>> Or are there other reasons to exclude them from the search?

Sometimes the admin directory is so large (esp number of files) that the
performance impact is very significant (can be slower by more than a factor
of 2).


        Stefan