rgrep buglet

rgrep buglet

[joakim]

I have this function:

(defun planner-grep-unfinished-tasks ()
  (interactive)
  ;;oddly, grep must have been run once interactively for this to work
  (let ((grep-find-template (concat grep-find-template "|sort -t: -k3")))
    (rgrep "#[ABCDEFGH][0-9]* *_" "*.muse" "~/Plans")))

Do the following in a fresh emacs:
m-x  planner-grep-unfinished-tasks RET

you get:

-*- mode: grep; default-directory: "~/Plans/" -*-
Grep started at Tue Aug 11 16:25:43

|sort -t: -k3
/bin/bash: -c: line 0: syntax error near unexpected token `|'
/bin/bash: -c: line 0: `|sort -t: -k3'

Grep exited abnormally with code 2 at Tue Aug 11 16:25:43

But if I 1st do:

m-x rgrep RET c-g

it works fine. Why is this?

-- 
Joakim Verona

Re: rgrep buglet

[Glenn Morris]

joakim@verona.se wrote:

[grep does not work if the first call is non-interactive]

Probably because of `grep-compute-defaults' only being run in the
interactive case. So if you manually call `grep-compute-defaults', it
might work.

As per the comment in bug#3343, this idea may need re-thinking.

Re: rgrep buglet

[Lennart Borgman]

On Tue, Aug 11, 2009 at 9:40 PM, Glenn Morris<rgm@gnu.org> wrote:
> joakim@verona.se wrote:
>
> [grep does not work if the first call is non-interactive]
>
> Probably because of `grep-compute-defaults' only being run in the
> interactive case. So if you manually call `grep-compute-defaults', it
> might work.
>
> As per the comment in bug#3343, this idea may need re-thinking.


Is there any reason not to recompute them every time if the
defcustom's have been changed? Is not this was users expect? (I have
seen several questions on this.)

Re: rgrep buglet

[Andreas Schwab]

Lennart Borgman <lennart.borgman@gmail.com> writes:

> Is there any reason not to recompute them every time if the
> defcustom's have been changed?

They are, see grep-apply-setting.

Andreas.

-- 
Andreas Schwab, schwab@linux-m68k.org
GPG Key fingerprint = 58CA 54C7 6D53 942B 1756  01D3 44D5 214B 8276 4ED5
"And now for something completely different."