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

Obsolete instruction in SubmittingPatches?

From
Andrei Rybak <rybak.a.v@gmail.com>
Date
Feb 28, 2018, 21:31 UTC
Message-ID
<48b5c506-6882-81e2-35f6-7d040bfc6017@gmail.com>
Hello,

It seems to me that this part of Documentation/SubmittingPatches is not actual nowadays:

  After the list reached a consensus that it is a good idea to apply the
  patch, re-send it with "To:" set to the maintainer{1} and "cc:" the
  list{2} for inclusion.

From what I observe in the mailing list, most patches are sent with "To:" set to mailing list (or re-sent with increasing version) , as per previous paragraph in the guidelines [1]. Then, after the topic is reviewed and the [PATCH v<last>] series receives a thumbs up from a number of people, the maintainer--Junio C Hamano--replies with an email containing something along the lines of "This change is in good shape. Thanks, will queue.", which makes me think that the re-send is not actually needed.

Is this part of guidelines obsolete, or am I not understanding this correctly?

[1] "Send your patch with "To:" set to the mailing list, with "cc:"
    listing people who are involved in the area you are touching"

-- Best regards, Andrei Rybak

Next: Junio C Hamano
Message 1 of 4 in “Obsolete instruction in SubmittingPatches?”
  1. Andrei RybakFeb 28, 2018
  2. Junio C HamanoFeb 28, 2018
  3. Andrei RybakMar 1, 2018
  4. Junio C HamanoMar 1, 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.