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

Re: [PATCH] docs: clarify meaning of core.commentString=auto

From
Junio C Hamano <gitster@pobox.com>
Date
Mar 18, 2025, 17:15 UTC
Message-ID
<xmqqa59i45wc.fsf@gitster.g>
In-Reply-To
<Z9lcXR6sL3UWlL33@ugly>
Oswald Buddenhagen <oswald.buddenhagen@gmx.de> writes:
Show 6 quoted lines
>>before making a commit.  But "this makes it impossible" does not
>>quite convey that to casual readers.
>>
> no, i meant what i wrote: it makes it _literally_ impossible. it follows
> from the preceding sentence that _whatever_ is in the template will NOT
> be the comment char.

OK, it (i.e. the order in which things happen) would be a good thing to add to the explanation, to unconfuse readers who (incorrectly) guess that auto comment character is determined and then template is read, which is where my comment came from.

> reading through the thread of the original submission, the feature is a
> workaround for `commit -m` and `commit --amend` being inconsistent wrt.
> message washing.
Perhaps somebody can be talked into fixing it ;-)

With a clear explanation, I am OK if somebody wants to advocate to deprecate (and remove at Git 3.0 boundary) the "auto" support ;-)

Thanks.
Previous: Oswald BuddenhagenNext: Oswald Buddenhagen
Message 5 of 8 in “docs: clarify meaning of core.commentString=auto”
  1. docs: clarify meaning of core.commentString=autoOswald Buddenhagen, Mar 15, 2025
  2. Junio C HamanoMar 17, 2025
  3. Taylor BlauMar 17, 2025
  4. Oswald BuddenhagenMar 18, 2025
  5. Junio C HamanoMar 18, 2025
  6. Oswald BuddenhagenMar 19, 2025
  7. Phillip WoodMar 20, 2025
  8. Junio C HamanoMar 21, 2025

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.