GNU bug report logs - #20728
25.0.50; grep and grep-find templates should have a place holder for the --color argument

Previous Next

Package: emacs;

Reported by: Dmitry Gutov <dgutov <at> yandex.ru>

Date: Wed, 3 Jun 2015 23:44:01 UTC

Severity: normal

Found in version 25.0.50

Fixed in version 25.1

Done: Dmitry Gutov <dgutov <at> yandex.ru>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Dmitry Gutov <dgutov <at> yandex.ru>
Cc: tracker <at> debbugs.gnu.org
Subject: bug#20728: closed (25.0.50; grep and grep-find templates should
 have a place holder for the --color argument)
Date: Sat, 27 Jun 2015 21:12:02 +0000
[Message part 1 (text/plain, inline)]
Your message dated Sun, 28 Jun 2015 00:11:34 +0300
with message-id <558F1186.50303 <at> yandex.ru>
and subject line Re: bug#20728: 25.0.50; grep and grep-find templates should have a place holder for the --color argument
has caused the debbugs.gnu.org bug report #20728,
regarding 25.0.50; grep and grep-find templates should have a place holder for the --color argument
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs <at> gnu.org.)


-- 
20728: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=20728
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems
[Message part 2 (message/rfc822, inline)]
From: Dmitry Gutov <dgutov <at> yandex.ru>
To: bug-gnu-emacs <at> gnu.org
Subject: 25.0.50;
 grep and grep-find templates should have a place holder for the
 --color argument
Date: Thu, 04 Jun 2015 02:42:58 +0300
And fill it in dynamically during expansion.

This way, the caller can set grep-highlight-matches dynamically, without
recomputing grep defaults (which is expensive). This is particularly
relevant on MS-Windows, in semantic-symref-grep-use-template and
xref-collect-matches.


[Message part 3 (message/rfc822, inline)]
From: Dmitry Gutov <dgutov <at> yandex.ru>
To: Juri Linkov <juri <at> linkov.net>
Cc: 20728-done <at> debbugs.gnu.org
Subject: Re: bug#20728: 25.0.50; grep and grep-find templates should have a
 place holder for the --color argument
Date: Sun, 28 Jun 2015 00:11:34 +0300
Version: 25.1

On 06/12/2015 02:20 AM, Juri Linkov wrote:

> Yes, it's out of the question that we should keep the original letters.

I've just pushed a commit that reuses <C> for both -i and --color 
(<C>ase and <C>olor, we can still pretend it's mnemonical), simply 
because there are templates out there with <C> in them, which do not 
include "--color".

If anyone really wants to change it to <O>, be my guest.

> I wonder how many third-party code uses the templates with <N>
> that it needs to be documented.

I haven't found any. OTOH, it doesn't hurt anyone either. Maybe the 
right change would be to make the code use it, actually. Are there use 
cases for changing the value of null-decide dynamically?

And speaking of needs, what is the use case for "--color" without 
"always"? Instead of special-casing Windows and DOS, why don't we always 
use `always' for coloring?


This bug report was last modified 10 years and 4 days ago.

Previous Next


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