[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: PATCH: vc-git-grep: force git-grep to disable color and pager

From: Mitchel Humpherys
Subject: Re: PATCH: vc-git-grep: force git-grep to disable color and pager
Date: Mon, 30 Sep 2013 20:51:19 -0700

On Wed, Sep 25, 2013 at 10:17 PM, Mitchel Humpherys
<address@hidden> wrote:
> On Wed, Sep 25, 2013 at 10:08 AM, Davis Herring <address@hidden> wrote:
>>> This commit adds some command line options to the invocation of `git'
>>> in `vc-git-grep' so that git disables all coloring and paging. Without
>>> this, I get a warning about my "terminal" not being fully functional
>>> and git-grep attempts to use a pager (which means the git-grep process
>>> never finishes):
>>>     git grep -n -e stuff -- *
>>>     WARNING: terminal is not fully functional
>>>      CREDITS:14:D: Alpha systems hacking, IPv6 and other network related 
>>> stuff
>>>     <snip>
>>>     :  <-- pager waiting for me to do something
>> All this (including the warning) is the use of less, not the coloring
>> (although git might in theory complain if you try to use coloring with
>> no TERM variable or so).  I'd recommend using --no-pager but trying to
>> interpret the color rather than suppressing it.
>> Davis
> You're absolutely right. I've removed the --no-color flag in this
> version and confirmed that highlighting works in the *grep* results
> buffer.

Any other comments on this? What's the next step towards merging? It's
quite a trivial patch but without it `vc-git-grep' is pretty much

"if i had more time i would have written a shorter letter."

reply via email to

[Prev in Thread] Current Thread [Next in Thread]