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

Re: [PATCH/RFC] ignore unknown color configuration

From
Junio C Hamano <gitster@pobox.com>
Date
Dec 16, 2009, 01:25 UTC
Message-ID
<7v3a3bwvzj.fsf@alter.siamese.dyndns.org>
In-Reply-To
<7vzl5mjndi.fsf@alter.siamese.dyndns.org>
Junio C Hamano <gitster@pobox.com> writes:
Show 18 quoted lines
> Jeff King <peff@peff.net> writes:
>
>> On Sat, Dec 12, 2009 at 01:45:45PM -0800, Junio C Hamano wrote:
>>
>>> This is a sane thing to do, as "slot" is part of the name of the variable,
>>> and we generally do not warn upon seeing a misspelled variable name (it
>>> makes it worse that "func" is not even misspelled but merely unknown to
>>> older version of git in your scenario).
>>> 
>>> On the other hand, I suspect that most people would apprecfiate if their
>>> git pointed out "diff.color.finc?  What do you mean?"  before they waste
>>> 30 minutes wondering why the new feature in 1.6.6 does not work for them.
>>
>> I would be more sympathetic to that user if this weren't the _only_ set
>> of variables with this property. They don't get warned for diff.externel
>> or color.show-branch.
>
> True and fair enough.  Let's have this in 1.6.6 then.

Actually I think we should have this in 1.6.5.X as well for it to be useful. Am I mistaken?

Previous: Junio C HamanoNext: Jeff King
Message 5 of 8 in “ignore unknown color configuration”
  1. ignore unknown color configurationJeff King, Dec 12, 2009
  2. Junio C HamanoDec 12, 2009
  3. Jeff KingDec 12, 2009
  4. Junio C HamanoDec 14, 2009
  5. Junio C HamanoDec 16, 2009
  6. Jeff KingDec 16, 2009
  7. Junio C HamanoDec 16, 2009
  8. Jeff KingDec 16, 2009

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.