<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Jun 25, 2014 at 11:13 AM, Reid Kleckner <span dir="ltr"><<a href="mailto:rnk@google.com" target="_blank">rnk@google.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Can you provide some data to support this by comparing commits with phab URLs in them with the llvm-commits archive?<div>

<br></div><div>It may be that llvm-commits is properly forwarding the review mail, but it's getting caught in people's spam filters.  I've personally had problems with this.</div>
</div><div class="gmail_extra"><br></div></blockquote><div><br></div><div>There was certainly a problem some time last week or the one before, wherein Phab wasn't forwarding emails to the mailing lists for many hours, and at some point flushed them all out. We had a number of outstanding comments that didn't appear anywhere until the flushing, and we checked the spam folders. It wasn't that - the emails were just delayed.</div>

<div><br></div><div>Eli</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="gmail_extra"><br><div class="gmail_quote"><div class="">

On Wed, Jun 25, 2014 at 10:44 AM, Alp Toker <span dir="ltr"><<a href="mailto:alp@nuanti.com" target="_blank">alp@nuanti.com</a>></span> wrote:<br></div><div><div class="h5"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


For whatever reason, patches posted to the Phabricator website still aren't being sent to the mailing list, making it difficult for us to review them.<br>
<br>
I've raised this issue a couple of times in the last few weeks.<br>
<br>
In practice this has a detrimental effect to the development workflow because it means that code is being seen only by a small group of individuals who have web accounts. The code isn't hitting llvm-commits or cfe-commits where the majority of code maintainers use the mailing lists for review.<br>



<br>
At this point I think Phabricator should be disabled and patches should be send to the mailing lists *until* the technical issue is confirmed resolved.<br>
<br>
It's really uncool that code is entering ToT through this back-channel -- I appreciate that it might not be intentional, but every single patch that gets committed this way is a real problem for the project.<span><font color="#888888"><br>



<br>
Alp.<br>
<br>
-- <br>
<a href="http://www.nuanti.com" target="_blank">http://www.nuanti.com</a><br>
the browser experts<br>
<br>
______________________________<u></u>_________________<br>
LLVM Developers mailing list<br>
<a href="mailto:LLVMdev@cs.uiuc.edu" target="_blank">LLVMdev@cs.uiuc.edu</a>         <a href="http://llvm.cs.uiuc.edu" target="_blank">http://llvm.cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev" target="_blank">http://lists.cs.uiuc.edu/<u></u>mailman/listinfo/llvmdev</a><br>
</font></span></blockquote></div></div></div><br></div>
<br>_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:LLVMdev@cs.uiuc.edu">LLVMdev@cs.uiuc.edu</a>         <a href="http://llvm.cs.uiuc.edu" target="_blank">http://llvm.cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev</a><br>
<br></blockquote></div><br></div></div>