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

Re: [PATCH] config: deprecate using "" as boolean value false.

From
Junio C Hamano <gitster@pobox.com>
Date
Feb 11, 2008, 08:21 UTC
Message-ID
<7vskzzaoop.fsf@gitster.siamese.dyndns.org>
In-Reply-To
<20080211082216.e9212310.chriscool@tuxfamily.org>

Why do you call this a "fix", when the only effect of it is breaking people's existing setups? Wouldn't a set of proper fixes to the config parsing callback functions solve all the issues without having any downsides?

I think you are wasting both of our times. You could be doing the real fix instead. Or I could be spending time on it, instead of repeating to you how mistaken this approach is.

So far I've seen and queued only two usable fixes since that [Janitor] request. Maybe I should have ignored all this discussion in this thread and spent my time on doing the Janitorial work. The reason I joined the thread was that I hoped that I could save people like _you_ from wasting _your_ time in the discussion, instead of making better use of it by doing audits and fixes.

Grrr.
Previous: Christian CouderNext: Junio C Hamano
Message 2 of 4 in “config: deprecate using "" as boolean value false.”
  1. config: deprecate using "" as boolean value false.Christian Couder, Feb 11, 2008
  2. Junio C HamanoFeb 11, 2008
  3. Junio C HamanoFeb 11, 2008
  4. teach config parsers to diagnose missing valuesJunio C Hamano, Feb 11, 2008

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.