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

Re: 'git format-patch <ref> --summary' seems to be turning off the stat

From
Kaartic Sivaraam <kaartic.sivaraam@gmail.com>
Date
Dec 18, 2017, 18:44 UTC
Message-ID
<41f53071-185d-58a5-499c-d93208aa70fc@gmail.com>
In-Reply-To
<xmqqlghzyj9q.fsf@gitster.mtv.corp.google.com>
On Tuesday 19 December 2017 12:07 AM, Junio C Hamano wrote:
Show 17 quoted lines
> Kaartic Sivaraam <kaartic.sivaraam@gmail.com> writes:
> 
>> Note: I do see that "--summary" is a diff-option but does that mean we
>> should't be printing stat information in the patch when the user
>> didn't mention "--no-stat"? Yeah, "git format-patch <ref> --summary
>> --stat" does bring back the stat. --- Kaartic
> 
> The old design decision shared across "log" family of commands [*1*]
> wrt to these "how and what are shown" options is that they started
> without any default set of options and the user gave all options (if
> they want both summary and stat, they gave them from the command
> line); over time they gained a set of built-in default options that
> take effect only when the user gives no option.
> 
> So, yes, giving "--summary" will turn off the built-in ones that are
> "--summary --stat" (IIRC).
>

Is it worth documenting that built-in defaults are turned off when user gives other options? Not sure where, though?

-- 
Kaartic
Previous: Junio C Hamano
Message 3 of 3 in “'git format-patch <ref> --summary' seems to be turning off the stat”
  1. Kaartic SivaraamDec 16, 2017
  2. Junio C HamanoDec 18, 2017
  3. Kaartic SivaraamDec 18, 2017

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.