<div dir="auto">Hi,<div dir="auto">If i can drop my five cents, max assume good will. If it taints release in deadly way keep it in purgatory. If its not fatal, let it get released but announce bounty for fixing it. The commms might be failure either phabricator or email process might fail as its unclear which to use etc as it was revealed recently.<div dir="auto"><br></div><div dir="auto">Id say make sure your change request hits author of patch. If theres doubt or difference of opinions discuss etc but id guess, opensource is based on kindness and volunteering etc so id work on it in aforementioned way above.</div><div dir="auto"><br></div><div dir="auto">Hope that helps.</div><div dir="auto"><br></div><div dir="auto">Best regards and best wishes for llvm,</div><div dir="auto">Pawel kunio</div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">pon., 26.04.2021, 14:58 użytkownik James Henderson via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.org</a>> napisał:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>Hi all,</div><div><br></div><div>TL;DR - what should you do if a patch author fails to respond to post-commit review comments? I'm guessing the answer is revert after sufficient prompting, but how much is sufficient prompting?</div><div><br></div><div>Background:<br></div><div><br></div><div>I was not too long ago involved in reviewing a patch on Phabricator (<a href="https://reviews.llvm.org/D95638" target="_blank" rel="noreferrer">https://reviews.llvm.org/D95638</a>). I made several comments, and some of these were addressed. Unfortunately, I didn't have the time to come back to the review at that time, and one or two others made a couple of review comments themselves. Eventually, after a couple of pings by the patch author, one of the people on the review approved the patch, and it was committed. About a week later, one of my colleagues noticed a bug in the patch, which prompted me to take another look at it. At this point, I noticed that the patch had landed without several of my comments being addressed; the patch in particular had (in my opinion) insufficient testing. I posted these comments on the Phabricator review page for the patch, asking for them to be addressed. After taking time off for a couple of weeks, I noticed that the patch author hadn't responded to my comments, so I prodded them again. A week later and there's still been no response.</div><div><br></div><div>What should I do at this point? I can of course revert the patch (and any subsequent patch that relied on it). Before taking that action though, I wanted to make sure this was the appropriate approach.</div><div><br></div><div>Thanks in advance for advice.</div><div><br></div><div>James<br></div></div>
_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org" target="_blank" rel="noreferrer">llvm-dev@lists.llvm.org</a><br>
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br>
</blockquote></div>