On 07/24/18 14:16, Marvin Häuser wrote:
> Hey Laszlo,
> 
>> -----Original Message-----

>> (1) Please post patch sets using a cover letter message:
>>
>> git config format.coverletter     true
>> git config format.numbered        true
>> git config sendemail.chainreplyto false
>> git config sendemail.thread       true
>>
>> Otherwise the individual patches in the series will get intermixed with other
>> messages / threads on the mailing list (and in people's inboxes), if the
>> recipient uses a threaded view in their MUA (which most people should do --
>> patches and discussions are very hard to follow otherwise).
> 
> Is V2 fine in this aspect? Outlook has never threaded multiple parts of a 
> patch series for me but only replies to their original message, hence I did 
> not notice something was off.

It doesn't look right to me, looking at the following patches patches:

[edk2] [PATCH v2 0/2] Introduce UEFI PI 1.5 MM PPI.
[edk2] [PATCH v2 1/2] MdePkg: Add PI 1.5 MM PPI declarations.
[edk2] [PATCH v2 2/2] MdeModulePkg: Change SMM PPIs to shim MM PPIs.

* v2 0/2:

Message-ID:
<vi1pr0801mb179040ba004eb739ce6aa07a80...@vi1pr0801mb1790.eurprd08.prod.outlook.com>

* v2 1/2:

Message-ID:
<vi1pr0801mb1790297b718346971dd0a30a80...@vi1pr0801mb1790.eurprd08.prod.outlook.com>

References: <cover.1532434029.git.marvin.haeu...@outlook.com>

In-Reply-To: <cover.1532434029.git.marvin.haeu...@outlook.com>

* v2 2/2:

Message-ID:
<vi1pr0801mb1790937d22054bf7815580ce80...@vi1pr0801mb1790.eurprd08.prod.outlook.com>

References: <cover.1532434029.git.marvin.haeu...@outlook.com>

In-Reply-To: <cover.1532434029.git.marvin.haeu...@outlook.com>


Here's what I think happened: the original message-id (formatted by your
local tooling) for the cover letter was

<cover.1532434029.git.marvin.haeu...@outlook.com>

Accordingly, your local tools placed this message ID *correctly* into
both "In-Reply-To" and "References" headers of the actual patches #1 and
#2. This is the proper way to ensure threading.

Unfortunately, outlook.com's SMTP server seems to have overwritten your
Message-ID, on the cover letter (and the rest of the emails too),
despite your tools generating and embedding Message-ID in the first
place. Obviously this breaks threading. This is a bug in outlook.com's
SMTP service, most likely.

Can you open a support ticket with them, or else use something else for
posting patches than outlook.com?

Thanks,
Laszlo
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel

Reply via email to