GNU bug report logs - #23590
25.0.94; Errors in default lgrep command

Previous Next

Full log

View this message in rfc822 format

From: Eli Zaretskii <eliz <at> gnu.org>
To: Lars Ingebrigtsen <larsi <at> gnus.org>
Cc: john.b.mastro <at> gmail.com, 23590 <at> debbugs.gnu.org, agrambot <at> gmail.com
Subject: bug#23590: 25.0.94; Errors in default lgrep command
Date: Wed, 07 Oct 2020 11:17:37 +0300

> From: Lars Ingebrigtsen <larsi <at> gnus.org>
> Cc: john.b.mastro <at> gmail.com,  Eli Zaretskii <eliz <at> gnu.org>,
>   23590 <at> debbugs.gnu.org
> Date: Wed, 07 Oct 2020 05:41:56 +0200
> 
> Lars Ingebrigtsen <larsi <at> gnus.org> writes:
> 
> > There was discussion here before the patch was posted, but then no
> > follow-up.  I've respun the patch for Emacs 28 -- does anybody have any
> > comments?
> 
> There were no comments in a month, and the patch makes sense to me, so
> I've applied it to Emacs 28.

I'm sorry to not have chimed earlier, but I think this change goes too
far.  Now the default "M-x grep" command includes --directories=skip
if the user's Grep program supports that.  This is not a good idea for
the general-purpose Grep commands.  For example, if the user adds to
the default command "-R foo" somewhere before the --directories=skip
part, the command will say "no matches", which would be a surprise at
best, and at worst could mislead the user.

The original bug report was about "M-x lgrep".  Can we make this
change affect only that command?

In any case, this is a user-visible change, so it should be called out
in NEWS.

Thanks.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.