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

Re: [PATCH v3] bisect: report the found commit with "show"

From
Jeff King <peff@peff.net>
Date
Apr 16, 2024, 05:13 UTC
Message-ID
<20240416051358.GB1745631@coredump.intra.peff.net>
In-Reply-To
<CAPig+cSY3vp6V=4SWmxyCi+7QY74eGnzPM6bu5LU5j00n8-j5g@mail.gmail.com>
On Mon, Apr 15, 2024 at 05:33:29PM -0400, Eric Sunshine wrote:
Show 13 quoted lines
> > Curious how you trimmed the trailers from the submitted patch ;-)
> >
> > Although we do not use Cc: in this project, we do recommend use of
> > the "Reported-by" trailer in Documentation/SubmittingPatches.
> 
> Sorry if that caused any confusion. I wasn't trying to make some sort
> of implicit suggestion to the patch author (such as "don't use Cc:
> trailers"). Rather, to ease the reading burden for others, I
> habitually trim off chunks of the email which aren't relevant to the
> portions about which I'm commenting. So, although I always leave the
> Signed-off-by: intact, I often trim off other trailers, sometimes even
> parts of the commit message (replacing with "[...]"), and, of course,
> entire chunks of the patch body.

FWIW, as a reader I very much appreciate when people trim their quotes to the most relevant bits.

-Peff
Previous: Eric SunshineNext: Peter Krefting
Message 5 of 6 in “bisect: report the found commit with "show"”
  1. bisect: report the found commit with "show"Peter Krefting, Apr 13, 2024
  2. Eric SunshineApr 14, 2024
  3. Junio C HamanoApr 15, 2024
  4. Eric SunshineApr 15, 2024
  5. Jeff KingApr 16, 2024
  6. Peter KreftingApr 16, 2024

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.