On 7/6/15, Jeff Law <l...@redhat.com> wrote:
> On 07/05/2015 04:58 PM, Eric Gallager wrote:
>> I was just matching the code that was already used there... should the
>> lines to ignore the CVS and .svn folders be re-written into the style
>> you propose, too?
> Might as well have a consistent style.  Embedding them into the "find"
> ought to be marginally faster than a pipeline of fgrep processes.
>
> jeff
>

Okay, I tried embedding "! -name CVS/ ! -name .svn/" into the find
command, too, but
that just led to warnings such as:
find: warning: Unix filenames usually don't contain slashes (though
pathnames do).  That means that '-name `CVS/'' will probably evaluate
to false all the time on this system.  You might find the '-wholename'
test more useful, or perhaps '-samefile'.  Alternatively, if you are
using GNU grep, you could use 'find ... -print0 | grep -FzZ `CVS/''.
find: warning: Unix filenames usually don't contain slashes (though
pathnames do).  That means that '-name `.svn/'' will probably evaluate
to false all the time on this system.  You might find the '-wholename'
test more useful, or perhaps '-samefile'.  Alternatively, if you are
using GNU grep, you could use 'find ... -print0 | grep -FzZ `.svn/''.

I'm not really sure which of these alternatives to go with... cc-ing
the fixincludes maintainer to see which he wants to go with...

Reply via email to