William Casarin <jb55@jb55.com> writes: > LGTM, I have a similar patch here: id:20200908171536.20679-1-jb55@jb55.com So if I understood correctly the problem with that was that you were unable to run the tests and therefore couldn't confirm that you fixed the issue or even that there was an issue in the first place. Well I was able to build and test and I can confirm that there is an issue and that your commit fixes it. So it should be merged (though I would merge my own variant because of the extra comment ;-). Cheers, Jonas _______________________________________________ notmuch mailing list -- notmuch@notmuchmail.org To unsubscribe send an email to notmuch-leave@notmuchmail.org