[debbugs-tracker] bug#9414: closed (Use of 'grep-find-ignored-directories' exclude files from search.)

[GNU bug Tracking System]

Your message dated Mon, 05 Sep 2011 12:55:32 +0300
with message-id <address@hidden>
and subject line Re: bug#9414: Fixed patch.
has caused the GNU bug report #9414,
regarding Use of 'grep-find-ignored-directories' exclude files from search.
to be marked as done.

(If you believe you have received this mail in error, please contact
address@hidden)


-- 
9414: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=9414
GNU Bug Tracking System
Contact address@hidden with problems

--- Begin Message --- Subject: Use of 'grep-find-ignored-directories' exclude files from search. Date: Thu, 01 Sep 2011 00:44:11 +0300 User-agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:6.0.1) Gecko/20110830 Thunderbird/6.0.1

GNU Emacs 23.3.1 (i386-mingw-nt6.1.7601) of 2011-03-10 on 3249CTO

If set:

  (add-to-list 'grep-find-ignored-directories "build" t)

and try search M-x rgrep in hierarchy:

  build/xxx
  src/yyy
  src/build
  build

  $ mkdir  build src;
  $ touch  build/xxx src/yyy src/build

'find' part of 'rgrep' print only (I simplify expression of 'rgrep'):

  $ find . '(' -path '*/build' -o -path '*/lib' ')' \
     -prune -o  -type f  -print
./src/yyy

Correct one is:

  $ find . '(' -type d -path '*/build' -o -path '*/lib' ')' \
     -prune -o  -type f  -print
./src/build
./src/yyy

So any dir added to 'grep-find-ignored-directories' also affected to
*regular* file names...

I previously file bug to findutils, but I am wrong:

  http://savannah.gnu.org/bugs/?34159

This is Emacs bug.

I suggest fix in patch.

grep-find-ignored-directories.patch
Description: Text document

--- End Message ---

> --- Begin Message ---
>
>
>
> Subject: 
>
> Re: bug#9414: Fixed patch.
>
>
>
>
> Date: 
>
> Mon, 05 Sep 2011 12:55:32 +0300
>
>
>
>
> User-agent: 
>
> Gnus/5.13 (Gnus v5.13) Emacs/24.0.50 (x86_64-pc-linux-gnu)
>
>
>
>
> > Previous patch is wrong (I miss 'find' options priority).
> >
> > This one right (more priestly tested and work fine).
>
> Thank you.  This patch is now committed.
>
>
> --- End Message ---