<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Jul 21, 2020 at 11:29 AM Mehdi AMINI <<a href="mailto:joker.eph@gmail.com">joker.eph@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Jul 21, 2020 at 11:07 AM David Blaikie <<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div><a class="gmail_plusreply" id="gmail-m_899660881811785255gmail-m_-5989698225186102693plusReplyChip-0" href="mailto:joker.eph@gmail.com" target="_blank">+Mehdi AMINI</a> who's taking some (shared?) ownership of Phabricator these days.<br><br>Mehdi - was Phab updated recently (such that we might've picked up new semantics)?</div></div></blockquote><div><br></div><div>No: I upgraded the hardware and the OS, but not Phab itself yet.</div><div><br></div><div>I have a test instance running with an upgraded Phab though, it may have been sending duplicate emails in the last day or two when I didn't notice I had the email daemon running.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Jul 21, 2020 at 4:25 AM Jay Foad via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Has anyone else noticed Phabricator sending emails saying:<br>
  This revision was not accepted when it landed; it landed in state<br>
"Needs Review".<br>
when the review clearly has been accepted by someone?<br>
<br>
Some recent examples:<br>
<a href="https://reviews.llvm.org/D83952" rel="noreferrer" target="_blank">https://reviews.llvm.org/D83952</a></blockquote></div></div></blockquote><div><br></div><div>Seems like this one closed as expected without the message? <a href="http://lists.llvm.org/pipermail/llvm-commits/Week-of-Mon-20200720/808734.html" target="_blank">http://lists.llvm.org/pipermail/llvm-commits/Week-of-Mon-20200720/808734.html</a></div></div></div></div></div></div></blockquote><div><br>In my inbox I have two emails for that review.<br>(though, also, on the commits list, I do see this: <a href="http://lists.llvm.org/pipermail/llvm-commits/Week-of-Mon-20200720/808735.html">http://lists.llvm.org/pipermail/llvm-commits/Week-of-Mon-20200720/808735.html</a> - though my mail client (gmail)  is rendering this one as in the same thread/doesn't seem to show the "[Differential]" prefix in the subject - not sure what's going on there, usually gmail is /too/ ready to group mails by actual subject text, rather than by thread ids in the email hedaers... )<br> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<a href="https://reviews.llvm.org/D80116" rel="noreferrer" target="_blank">https://reviews.llvm.org/D80116</a></blockquote></div></div></blockquote><div><br></div><div>Same here: <a href="http://lists.llvm.org/pipermail/llvm-commits/Week-of-Mon-20200720/808778.html" target="_blank">http://lists.llvm.org/pipermail/llvm-commits/Week-of-Mon-20200720/808778.html</a></div><div><br></div><div>Can you forward me the email you received for these revisions?</div></div></div></div></div></div></blockquote><div><br>Similarly, the separate "[Differential]" email seems to be unthreaded and contains the problematic "not accepted when it landed" text: <a href="http://lists.llvm.org/pipermail/llvm-commits/Week-of-Mon-20200720/808779.html">http://lists.llvm.org/pipermail/llvm-commits/Week-of-Mon-20200720/808779.html</a><br> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div class="gmail_quote"><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><div><br>Hard for me to tell what happened here. I wonder if it's related to making changes after review/before committing. While that's common in LLVM, I could imagine a review tool (especially if we picked up a newer version - as I don't think it's always had this behavior) might get fussy about that - perhaps it'd be configurable, so it'd say "this was committed with extra changes" but not "This was committed without review".<br><br>Do you have any examples that didn't have post-approval-pre-commit changes that still got this annotation about being committed without review?<br><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<a href="https://reviews.llvm.org/D81267" rel="noreferrer" target="_blank">https://reviews.llvm.org/D81267</a></blockquote><div><br></div><div>Last one seems more clear - one of the reviewers (rupprecht) still had the review marked "requires changes", so it was committed without closure on that</div></div></div></blockquote><div><br></div><div>Indeed this one shows the message: <a href="http://lists.llvm.org/pipermail/llvm-commits/Week-of-Mon-20200713/807554.html" target="_blank">http://lists.llvm.org/pipermail/llvm-commits/Week-of-Mon-20200713/807554.html</a></div><div></div><div><br></div><div>-- </div><div>Mehdi</div><div> </div></div></div></div></div></div>
</blockquote></div></div>