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

Re: [PATCH v2] doc/SubmittingPatches: correct subject guidance

From
Eric Sunshine <sunshine@sunshineco.com>
Date
Nov 8, 2017, 14:10 UTC
Message-ID
<CAPig+cRaygrQ9_J1UHr_rynPsUn2J0--RHRvP2mFQbVQhWEm-Q@mail.gmail.com>
In-Reply-To
<20171108134752.214056-1-adam@dinwoodie.org>
On Wed, Nov 8, 2017 at 8:47 AM, Adam Dinwoodie <adam@dinwoodie.org> wrote:
Show 27 quoted lines
> The examples and common practice for adding markers such as "RFC" or
> "v2" to the subject of patch emails is to have them within the same
> brackets as the "PATCH" text, not after the closing bracket.  Further,
> the practice of `git format-patch` and the like, as well as what appears
> to be the more common pratice on the mailing list, is to use "[RFC
> PATCH]", not "[PATCH/RFC]".
>
> Update the SubmittingPatches article to match.
>
> Signed-off-by: Adam Dinwoodie <adam@dinwoodie.org>
> ---
> diff --git a/Documentation/SubmittingPatches b/Documentation/SubmittingPatches
> @@ -184,12 +184,14 @@ lose tabs that way if you are not careful.
>  It is a common convention to prefix your subject line with
>  [PATCH].  This lets people easily distinguish patches from other
> -e-mail discussions.  Use of additional markers after PATCH and
> -the closing bracket to mark the nature of the patch is also
> -encouraged.  E.g. [PATCH/RFC] is often used when the patch is
> -not ready to be applied but it is for discussion, [PATCH v2],
> -[PATCH v3] etc. are often seen when you are sending an update to
> -what you have previously sent.
> +e-mail discussions.  Use of markers in addition to PATCH within
> +the brackets to describe the nature of the patch is also
> +encouraged.  E.g. [RFC PATCH] is often used when the patch is not
> +ready to be applied but it is for discussion, and can be added
> +with the `--rfc` argument to `git format-patch` or `git
> +send-email`, while [PATCH v2], [PATCH v3] etc.  are often seen

It has become a bit of a run-on sentence, but aside from that and the unnecessary extra whitespace between "etc." and "are", it looks good to me.

> +when you are sending an update to what you have previously sent,
> +and can be added with the `-v <n>` arguments to the same commands.
Previous: Adam DinwoodieNext: Adam Dinwoodie
Message 3 of 8 in “doc/SubmittingPatches: correct subject guidance”
  1. doc/SubmittingPatches: correct subject guidanceAdam Dinwoodie, Nov 8, 2017
  2. doc/SubmittingPatches: correct subject guidanceAdam Dinwoodie, Nov 8, 2017
  3. Eric SunshineNov 8, 2017
  4. Adam DinwoodieNov 9, 2017
  5. doc/SubmittingPatches: correct subject guidanceAdam Dinwoodie, Nov 9, 2017
  6. Eric SunshineNov 9, 2017
  7. doc/SubmittingPatches: correct subject guidanceAdam Dinwoodie, Nov 10, 2017
  8. Josh TriplettNov 10, 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.