git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH] Use --no-color option on git log commands.

From
Junio C Hamano <gitster@pobox.com>
Date
Nov 28, 2007, 04:45 UTC
Message-ID
<7vr6ib9dwx.fsf@gitster.siamese.dyndns.org>
In-Reply-To
<474C60FA.4040302@wanadoo.fr>
Pascal Obry <pascal.obry@wanadoo.fr> writes:
Show 13 quoted lines
> Junio C Hamano a écrit :
>> The patch is good as belt-and-suspender, thanks.
>
> Ok.
>
>> But I suspect that we should make 'true' to mean 'auto' someday in
>> git_config_colorbool().  Crazy people can set 'always' if they really
>> wanted to, but most normal people would not want color unless the output
>> goes to the terminal, I would think.
>
> I definitely agree. I add it set to true, using auto instead I do not
> have the problem. Anyway I still think that it is good to apply my patch
> to completely avoid such issues.
Yes, that is what I said.

Except that the patch is severely whitespace damaged, and the message lack a sign-off.

I fixed them up by hand, so no need to resend.
Previous: Pascal ObryNext: Junio C Hamano
Message 4 of 10 in “Use --no-color option on git log commands.”
  1. Use --no-color option on git log commands.Pascal Obry, Nov 26, 2007
  2. Junio C HamanoNov 26, 2007
  3. Pascal ObryNov 27, 2007
  4. Junio C HamanoNov 28, 2007
  5. "color.diff = true" is not "always" anymore.Junio C Hamano, Nov 28, 2007
  6. Johannes SchindelinNov 28, 2007
  7. Jeff KingNov 28, 2007
  8. Junio C HamanoDec 1, 2007
  9. Jeff KingDec 1, 2007
  10. Junio C HamanoDec 1, 2007

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.