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

Re: [PATCH] format-patch: use --compact-summary instead of --summary

From
Duy Nguyen <pclouds@gmail.com>
Date
Mar 17, 2018, 08:08 UTC
Message-ID
<CACsJy8DBDqin3xYk70A2HxrZ6AYJhd0V5dj98Ni+j8t0vdNGyQ@mail.gmail.com>
In-Reply-To
<xmqqlgerzojl.fsf@gitster-ct.c.googlers.com>
On Fri, Mar 16, 2018 at 10:41 PM, Junio C Hamano <gitster@pobox.com> wrote:
Show 12 quoted lines
> Nguyễn Thái Ngọc Duy  <pclouds@gmail.com> writes:
>
>> For reviewers, a couple fewer lines from diffstat (either cover letter
>> or patches) is a good thing because there's less to read.
>
> I can certainly accept that there may be some reviewers who share
> that view, but me personally, I would rather have them outside the
> main diffstat section, so addition and removal of files that are
> rarer events do stand out.
>
> I guess people would not mind an option to use it, but they can
> already say "git format-patch --compact-summary $upstream..", so...
Good point. I can't negate --summary though. But that calls for another patch.
-- 
Duy
Previous: Junio C Hamano
Message 3 of 3 in “format-patch: use --compact-summary instead of --summary”
  1. format-patch: use --compact-summary instead of --summaryNguyễn Thái Ngọc Duy, Mar 16, 2018
  2. Junio C HamanoMar 16, 2018
  3. Duy NguyenMar 17, 2018

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.