2018-06-05 1:26 GMT+02:00 Thomas Gleixner <[email protected]>: > On Mon, 4 Jun 2018, Thomas Gleixner wrote: > >> The Message-ID extraction from the cover letter fails because the message >> generator uses 'Message-Id' while the extraction uses 'Message-ID'. formail >> did not care, but the extraction sed magic cares. As a consequence >> threading of the mails is broken. >> >> Use 'Message-ID' consistently all over the place. >> >> Fixes: 1d659bebaf3d ("quilt/mail: Remove procmail dependency") >> Signed-off-by: Thomas Gleixner <[email protected]> > > I noticed that the patch has been applied already. Thanks for that. > > Though I disagree with the rewritten changelog: > > (The mail threading broken by commit 1d659beb "quilt/mail: Remove > procmail dependency" was already fixed by commit 360b85e1 "mail: Fix > patch set threading".) > > That's blantantly wrong. commit 360b85e1 fixes absolutely nothing because > it merily adds -n to the sed invocation which does not at all fix the > uppercase/lowercase mismatch of the supplied header search string > 'Message-ID' and the header 'Message-Id' in the mail generator which > creates the cover letter from which the message id should be extracted.
Surely we must be talking about different commits? I've even verified that commit 360b85e1f6b6d1aff5ada942fcee816e1ad7a13c fixes message threading, with or without your "Message-ID" fix. Thanks, Andreas _______________________________________________ Quilt-dev mailing list [email protected] https://lists.nongnu.org/mailman/listinfo/quilt-dev
