<div dir="auto"><div><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Jun 24, 2022, 12:32 PM Jeremy White <<a href="mailto:jwhite@codeweavers.com">jwhite@codeweavers.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">So Zeb's evil plan to break my mailing list script worked!<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">😱</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
The moderately fragile bit that parses the patches relies on a regex <br>
matching '^From '.  She included a comment in her code referencing <br>
another patch; a 'From <sha> here'.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Doesn't the mbox format auto-escape it as `>From `? Maybe it's in another format or already parsed...</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
That caused it to crash after sending 3 out of 4 emails, and before it <br>
marked that MR as done.  So every time it woke up, it thought it needed <br>
to send that MR, would send 3, and then crash.<br>
<br>
I have, in theory, mitigated that with this change:<br>
   <a href="https://gitlab.winehq.org/winehq/tools/-/merge_requests/15" rel="noreferrer noreferrer" target="_blank">https://gitlab.winehq.org/winehq/tools/-/merge_requests/15</a><br>
<br>
Sorry for the over flowing mailbox.  Hopefully this patch set will get a <br>
*very* thorough review :-).<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Maybe I'm too late to the party, but a better way to fix this might be to remember the delivery state of each patch. Just a suggestion though; feel free to dismiss it 😅</div><div dir="auto"><br></div><div dir="auto">Also, since you particularly mentioned a *very* through review, a nit: in Python we usually use `is None` instead of `== None`. Both works, though.</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Cheers,<br>
<br>
Jeremy<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">-- </div><div dir="auto">Sincerely,</div><div dir="auto">Jinoh Kang</div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"></blockquote></div></div></div>