|
From: Florent R. <f.r...@fr...> - 2023-07-18 10:30:29
|
Julian Smith <ju...@op...> wrote: > Great, good to know you think it's ok. I've just pushed the fix. Thanks a lot! >> P.S.: your patch (the attachment) had all explanations plus your >> signature inside (yeah, I know the patch tool tolerates >> “trailing, uh... garbage”). :) > > I reckon that might be your email client? - i didn't attach the patch, > just included it inline. Ah, that's indeed what you did. Your mail is displayed as expected by default, but as I wrongly assumed your patch was an “inline attachment” (a standalone MIME part of the mail), I pressed 'K b' in Gnus in order to cause it to show a button for each MIME part, so as to allow me to save the patch without risking any whitespace problem. This is where it gets interesting, because after this 'K b' key sequence, Gnus pretended there were 3 parts in your mail, [1. text/plain], [2. text/x-patch] and [3. text/plain]. The second “part” is your patch + everything preceding the standards-conforming signature (the one started by dash-dash-space), and the third one is that standards-conforming signature. Not a big deal, but “interesting”. :) Regards -- Florent |