Re: git: abdcd967dc0c - main - =?utf-8?Q?Add new french b=C3=A9po keyboard layout (version 1.1rc2) normalized by F=Q?Add new french b=C3=A9po keyboard layout (version 1.1rc2) normalized by F=

From: Kristof Provost <kp_at_FreeBSD.org>
Date: Wed, 13 Apr 2022 07:17:51 UTC
On 12 Apr 2022, at 22:42, Dimitry Andric wrote:
> On 12 Apr 2022, at 21:29, Cy Schubert <Cy.Schubert@cschubert.com> wrote:
>>
>> In message <20220412190504.juclbpyc6xt3q76w@mutt-hbsd>, Shawn Webb writes:
>>>
>>> --domzta7c4zc4deqc
>>> Content-Type: text/plain; charset=utf-8
>>> Content-Disposition: inline
>>> Content-Transfer-Encoding: quoted-printable
>>>
>>> Um, What happened here?
>>
>> Looks like some garbage got inserted into the header causing some header
>> lines to be considered part of the body.
>
> Looks to me like the commit mailer can't handle it when there is no
> empty line between the body of the commit message and the "PR:",
> "Approved by:", etc headers.
>
I think we’ve seen that before, without issue.

My guess is that it’s a bug in the post-commit script that generates the e-mail.

The first line of the commit has utf-8 characters in it. Sending utf-8 in a subject line requires a =?charset? prefix. That was added here, but then the ‘git: hash - branch -‘ prefix was added afterwards.

Kristof