patchgrep - Patches: patch #883, patch to color filenames in grep...


You are not allowed to post comments on this tracker with your current authentification level.

patch #883: patch to color filenames in grep output

Submitted by:  Invalid User ID <Invalid User ID>
Submitted on:  Mon Dec 23 16:04:27 2002  
Category: NonePriority: 5 - Normal
Status: DuplicatePrivacy: Public
Assigned to: NoneOpen/Closed: Closed

Wed Jan 26 14:55:10 2005, comment #5:

OK, I closed this patch.
Obsoleted by patch #3644.

Stepan Kasal <kasal>
Project Member
Sat Jan 15 02:11:17 2005, comment #4:


Charles Levert <charles_levert>
Sat Jan 15 02:08:51 2005, comment #3:

I have implemented DIR_COLORS (plural) in
patch #3644 so it now completely obsoletes
and is a strict functional superset of this

Charles Levert <charles_levert>
Fri Jan 14 21:32:38 2005, comment #2:

I have submitted a new patch that does additional
colorizing more extensively (patch #3644).

Already, not only can lines of content be
prepended with the filename (-H), they can also
be with the line number (-n) and the octet
offset (-b). When context is used, groups of
lines can be separated with "--" on a single
line (which can be mistaken for content when
nothing is prepended as with -3h).

Moreover, with an additional new feature (-T or
--initial-tab), whitespace can be inserted
between a prepended item and its separator (";"
or "-").

All this highlights the fact that separators
should not be treated as being part of the
prepended item they follow, and so that they
merit their own color.

The new patch implements all this, but it
purposely leaves out a mechanism for user
configuration, for now. The approach for
this that is used in this patch #883, adding
one variable per new colorizable item, is
one option, but it can make for many variables.
Another solution to be considered would be to
reuse the single variable GREP_COLOR in a
backward compatible way so that it both supports
just the string for matches, and a new full
specification similar to what is done with
LS_COLORS (see dircolors(1)). This is up for
careful evaluation.

Charles Levert <charles_levert>
Fri Aug 15 09:29:45 2003, comment #1:

Duh! Sorry about that: my last comment should apply to a different patch.



No files currently attached


Depends on the following items: None found

Items that depend on this one: None found


CC list is empty


Do you think this task is very important?
If so, you can click here to add your encouragement to it.
This task has 0 encouragements so far.

Only logged-in users can vote.


Please enter the title of George Orwell's famous dystopian book (it's a date):



Follow 2 latest changes.

Date Changed By Updated Field Previous Value => Replaced By
Wed Jan 26 14:55:10 2005kasalStatusNone=>Duplicate

Back to the top

Powered by Savane 3.1-cleanup1