<div dir="ltr">Hi Ed,<div><br></div><div>I field <a href="https://urldefense.proofpoint.com/v2/url?u=https-3A__secure.phabricator.com_T8336&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=Mfk2qtn1LTDThVkh6-oGglNfMADXfJdty4_bhmuhMHA&m=QpkZnMD3eWte7upl9CJh74COv6x4iD9oQVA3gyE27ag&s=4Cgk6g_qdG-AALjpDAMWX0VUmX3LRiVffl4i8LCzFZI&e=">https://secure.phabricator.com/T8336</a> - feel free to add your requirements to that bug.<br><br><div class="gmail_quote">On Wed, May 27, 2015 at 8:39 PM Ed Maste <<a href="mailto:emaste@freebsd.org">emaste@freebsd.org</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 27 May 2015 at 12:29, Justin Bogner <<a href="mailto:mail@justinbogner.com" target="_blank">mail@justinbogner.com</a>> wrote:<br>
> Mehdi Amini <<a href="mailto:mehdi.amini@apple.com" target="_blank">mehdi.amini@apple.com</a>> writes:<br>
>> Hi Manuel,<br>
>><br>
>> I like Phabricator for code review much much more than emails. Let me know how<br>
>> I can help (I’m not afraid of PHP).<br>
><br>
> Chandler updated the llvm phabricator doc to point at what we're deploying:<br>
><br>
>   <a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__llvm.org_docs_Phabricator.html-23status&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=Mfk2qtn1LTDThVkh6-oGglNfMADXfJdty4_bhmuhMHA&m=QpkZnMD3eWte7upl9CJh74COv6x4iD9oQVA3gyE27ag&s=HSRe84f3_SLxGD04dp-M_x8o-n_8uovDDYmsGQ-d6G0&e=" target="_blank">http://llvm.org/docs/Phabricator.html#status</a><br>
><br>
> That'll lead you here:<br>
><br>
>   <a href="https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_r4nt_llvm-2Dreviews&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=Mfk2qtn1LTDThVkh6-oGglNfMADXfJdty4_bhmuhMHA&m=QpkZnMD3eWte7upl9CJh74COv6x4iD9oQVA3gyE27ag&s=hnlwZjMaqoPW44qfgu_zooBUdmE8mEsMp2fKLvikWHU&e=" target="_blank">https://github.com/r4nt/llvm-reviews</a><br>
>   <a href="https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_r4nt_phabricator&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=Mfk2qtn1LTDThVkh6-oGglNfMADXfJdty4_bhmuhMHA&m=QpkZnMD3eWte7upl9CJh74COv6x4iD9oQVA3gyE27ag&s=89YAkY7FdgJnVuifD6Kag9rwwTqd3jbRLpWdyBSHOfI&e=" target="_blank">https://github.com/r4nt/phabricator</a><br>
><br>
> And soon there'll be some bugs to squash in <a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__llvm.org_bugs&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=Mfk2qtn1LTDThVkh6-oGglNfMADXfJdty4_bhmuhMHA&m=QpkZnMD3eWte7upl9CJh74COv6x4iD9oQVA3gyE27ag&s=bMDWXJHpSMpb9J-qyZ43IdlVfBV1tpW44T-DqNlQCG4&e=" target="_blank">llvm.org/bugs</a> ;)<br>
<br>
On a related note we're using Phabricator in FreeBSD, with a similar<br>
workflow to LLVM and with a similar set of user opinions.  Some of our<br>
users really like web-based review and some can't stand it, and so<br>
we'll need to solve many of the same issues so that it's useful (or at<br>
least tolerable) to all.  We haven't yet made a lot of usability<br>
improvements, but our current set of changes can be found in our<br>
github repo here: <a href="https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_freebsd_phabricator&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=Mfk2qtn1LTDThVkh6-oGglNfMADXfJdty4_bhmuhMHA&m=QpkZnMD3eWte7upl9CJh74COv6x4iD9oQVA3gyE27ag&s=Nc5FQOQt9bIVp5QtkM7UIPK0SEjrdaExEP6s3Jv4r9Y&e=" target="_blank">https://github.com/freebsd/phabricator</a><br>
<br>
(As an aside, inspiration for setting up our Phabricator instance came<br>
from LLVM's.)<br>
<br>
_______________________________________________<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/mailman/listinfo/llvmdev</a><br>
</blockquote></div></div></div>