GNU bug report logs - #35326
26.1; vc-git-grep should pass -i like isearch

Previous Next

Package: emacs;

Reported by: Antoine Beaupré <anarcat <at> orangeseeds.org>

Date: Fri, 19 Apr 2019 18:33:02 UTC

Severity: minor

Tags: fixed

Found in version 26.1

Fixed in version 27.1

Done: Lars Ingebrigtsen <larsi <at> gnus.org>

Bug is archived. No further changes may be made.

Full log


Message #20 received at 35326 <at> debbugs.gnu.org (full text, mbox):

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: Juri Linkov <juri <at> linkov.net>
Cc: Antoine Beaupré <anarcat <at> orangeseeds.org>,
 35326 <at> debbugs.gnu.org
Subject: Re: bug#35326: 26.1; vc-git-grep should pass -i like isearch
Date: Tue, 25 Jun 2019 12:59:44 +0200
Juri Linkov <juri <at> linkov.net> writes:

> The confusion was caused by the original report mentioning `M-x grep'.
> In fact, `M-x grep' doesn't expand the template.  Only the following
> commands expand the template:
>
>   lgrep
>   rgrep
>   vc-git-grep
>
>> And should the vc-git version be more like grep or lgrep?
>
> vc-git-grep should be like other commands that expand the template,
> i.e. lgrep and rgrep, that the patch in the original report
> correctly achieves, so committing it would be an improvement
> since `git grep' supports most of grep options, including --color,
> -i and -w.

Makes sense.  I've now applied Antoine's patch to the trunk.

> PS: Regarding -w that I proposed previously, I could create
> a separate request, but here is a preliminary version:

[...]

> +                           (when grep-word-search
> +                             (push "-w" opts))

Hm...  I always just say \bfoo\b when I want the word-ish version, so I
don't know whether this really warrants a new command.

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no




This bug report was last modified 5 years and 330 days ago.

Previous Next


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