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

Re: Formatting problem send_mail in version 2.10.0

From
LFLarry Finger <larry.finger@lwfinger.net>
Date
Oct 11, 2016, 15:42 UTC
Message-ID
<45cfc4e5-c30e-19cb-ec3e-407ceb4e3ad5@lwfinger.net>
In-Reply-To
<vpqfuo3l4fl.fsf@anie.imag.fr>
On 10/11/2016 02:39 AM, Matthieu Moy wrote:
Show 29 quoted lines
> Jeff King <peff@peff.net> writes:
>
>> [+cc authors of b1c8a11, which regressed this case; I'll quote liberally
>>      to give context]
>>
>> On Mon, Oct 10, 2016 at 05:48:56PM -0400, Jeff King wrote:
>>
>>> I can't reproduce the problem with this simple setup:
>>>
>>> 	git init
>>> 	echo content >file && git add file
>>> 	git commit -F- <<-\EOF
>>> 	the subject
>>>
>>> 	the body
>>>
>>> 	Cc: Stable <stable@vger.kernel.org> [4.8+]
>
> Is this RFC2822 compliant (https://tools.ietf.org/html/rfc2822)? Not an
> expert of the norm, but my understanding is that you're allowed to use
> either "Name <addr@domain.com>" (name-addr) or addr@domain.com
> (addr-spec), and that comments are allowed within parenthesis like
> "Stable <stable@vger.kernel.org> (4.8+)".
>
> What is this [4.8+] supposed to mean?
>
> The guilty function is parse_mailboxes in perl/Git.pm. It should be
> rather easy to modify it but I need to understand the spec before I can
> try to implement anything.

That added information at the end is intended to be passed on to the stable group. In this case, the patch needs to be applied to kernel versions 4.8 and later.

Placing the info inside parentheses causes it to be dropped from the outgoing mail as follows:

(body) Adding cc: Stable <stable@vger.kernel.org> (4.8+) from line 'Cc: Stable 
<stable@vger.kernel.org> (4.8+)'
--snip--
Cc:     Stable <stable@vger.kernel.org>,
Larry
Previous: Matthieu MoyNext: Matthieu Moy
Message 7 of 24 in “Formatting problem send_mail in version 2.10.0”
  1. Larry FingerOct 10, 2016
  2. Jeff KingOct 10, 2016
  3. Jeff KingOct 10, 2016
  4. Larry FingerOct 10, 2016
  5. Jeff KingOct 10, 2016
  6. Matthieu MoyOct 11, 2016
  7. Larry FingerOct 11, 2016
  8. Matthieu MoyOct 11, 2016
  9. Larry FingerOct 12, 2016
  10. Matthieu MoyOct 12, 2016
  11. Larry FingerOct 12, 2016
  12. Matthieu MoyOct 12, 2016
  13. Larry FingerOct 12, 2016
  14. Matthieu MoyOct 12, 2016
  15. Larry FingerOct 12, 2016
  16. Junio C HamanoOct 12, 2016
  17. Jeff KingOct 12, 2016
  18. Matthieu MoyOct 13, 2016
  19. parse_mailboxes: accept extra text after <...> addressMatthieu Moy, Oct 13, 2016
  20. Kevin DaudtOct 13, 2016
  21. Matthieu MoyOct 13, 2016
  22. Matthieu MoyOct 13, 2016
  23. Junio C HamanoOct 14, 2016
  24. Jeff KingOct 11, 2016

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.