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

Re: [PATCHv2] Documentation/SubmittingPatches: Explain the rationale of git notes

From
Stefan Beller <sbeller@google.com>
Date
Dec 29, 2014, 23:21 UTC
Message-ID
<CAGZ79kZxsWZRdaQ-0_9Ou3y0KOruD0WVrC0Drkjgu=TbgUjiSA@mail.gmail.com>
In-Reply-To
<CAPig+cQgG5Ua-k1D8yKF5K+tfXeSB_wTN9W3m08F+K7AtKWDaw@mail.gmail.com>
On Mon, Dec 29, 2014 at 3:18 PM, Eric Sunshine <sunshine@sunshineco.com> wrote:
Show 23 quoted lines
> On Mon, Dec 29, 2014 at 12:42 PM, Stefan Beller <sbeller@google.com> wrote:
>> This adds more explanation of why you want to have the --notes option
>> given to git format-patch.
>>
>> Signed-off-by: Stefan Beller <sbeller@google.com>
>> ---
>> diff --git a/Documentation/SubmittingPatches b/Documentation/SubmittingPatches
>> index e3c942e..f42c607 100644
>> --- a/Documentation/SubmittingPatches
>> +++ b/Documentation/SubmittingPatches
>> @@ -177,7 +177,12 @@ message starts, you can put a "From: " line to name that person.
>>  You often want to add additional explanation about the patch,
>>  other than the commit message itself.  Place such "cover letter"
>>  material between the three dash lines and the diffstat. Git-notes
>> -can also be inserted using the `--notes` option.
>> +can also be inserted using the `--notes` option. If you are one
>> +of those developers who cannot write perfect code the first time
>> +and need multiple iterations of review and discussion, you may
>> +want to keep track of the changes between different versions of
>> +a patch using notes and then also use the `--notes` option when
>> +preparing the patch for submission.
>
> Perhaps this could be rephrased in a less derogatory fashion like this:

This wasn't meant to be derogatory at all. It was more of subtle humor. Sorry for the confusion.

If this is offensive in any way I'd rather go with your suggestion indeed.
Show 11 quoted lines
>
>     ...material between the three dash line and the diffstat.
>     For patches requiring multiple iterations of review and
>     discussion, an explanation of changes between each iteration can
>     be kept in Git-notes and inserted automatically following the
>     three dash line via `git format-patch --notes`.
>
>>  Do not attach the patch as a MIME attachment, compressed or not.
>>  Do not let your e-mail client send quoted-printable.  Do not let
>> --
>> 2.2.1.62.g3f15098
Previous: Eric SunshineNext: Junio C Hamano
Message 9 of 10 in “Documentation/SubmittingPatches: Explain the rationale of git notes”
  1. 1/2 Documentation/SubmittingPatches: Explain the rationale of git notesStefan Beller, Dec 18, 2014
  2. Documentation/SubmittingPatches: unify whitespace/tabs for the DCOStefan Beller, Dec 18, 2014
  3. 2/2 Documentation/SubmittingPatches: unify whitespace/tabs for the DCOStefan Beller, Dec 18, 2014
  4. Junio C HamanoDec 22, 2014
  5. Junio C HamanoDec 22, 2014
  6. Stefan BellerDec 29, 2014
  7. [PATCHv2] Documentation/SubmittingPatches: Explain the rationale of git notesStefan Beller, Dec 29, 2014
  8. Eric SunshineDec 29, 2014
  9. Stefan BellerDec 29, 2014
  10. Junio C HamanoDec 30, 2014

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.