<div dir="ltr">Hi Thiago,<div><br></div><div>CCing lldb-dev since I think these are great questions that I'm sure others watching the list may have them as well.<br><div class="gmail_extra"><br><div class="gmail_quote">
On Tue, Apr 8, 2014 at 8:41 PM, Thiago Farina <span dir="ltr"><<a href="mailto:tfarina@chromium.org" target="_blank">tfarina@chromium.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<div dir="ltr">Hi Todd,<div><br></div><div>I was surprised to notice that lldb is not using a codereview tool for his patch review process.</div><div><br></div><div>Why is that? No resources (infra and devs) to manage an installation of phabricator, rietveld or gerrit?</div>

<div><br></div></div></blockquote><div><br></div><div>Some people do indeed use Phabricator.  You can go here:</div><div><a href="http://reviews.llvm.org/">http://reviews.llvm.org/</a><br></div><div><br></div><div>It is not all that common, though. I think because of the process laid out here:</div>
<div><a href="http://llvm.org/docs/DeveloperPolicy.html">LLVM Developer Policy</a><br></div><div><br></div><div>which states a process for LLVM that involves sending patches to (for lldb) lldb-commits or lldb-dev, which doesn't specifically require (although it does support) a gerrit/Phabricator/etc. change list review site.</div>
<div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div></div><div>Also, I noticed that it is still using svn for its source control. I use git, so that is a barrier.</div>
<div><br></div></div></blockquote><div><br></div><div>You can pull code down from the git mirror, we do this all the time:</div><div><a href="http://llvm.org/git/llvm">http://llvm.org/git/llvm</a></div><div><div><a href="http://llvm.org/git/clang">http://llvm.org/git/clang</a></div>
</div><div><div><a href="http://llvm.org/git/lldb">http://llvm.org/git/lldb</a></div></div><div><br></div><div>I think it is documented <a href="http://llvm.org/docs/GettingStarted.html#git-mirror">here</a>.</div><div><br>
</div><div>If you get submit permissions, I find it way easier to submit into an svn tree since I've always found git-svn integration flaky, but some people do submit (via git-svn) and just work entirely out of git.  You have those options to consider.</div>
<div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div></div><div>Also it does not seem to have a CQ.</div>
</div></blockquote><div><br></div><div>CQ - can you define?  Do you mean a quality control system like the <a href="http://lab.llvm.org:8011/console">llvm buildbot system</a>?  It doesn't *appear* that we collectively look at them frequently, but I might be wrong on that.  Fortunately, people are quick to chime in on lldb-dev when there's an issue on their build.  Sometime in the future I plan on adding a Google Android buildbot and it will be spitting out nag emails when our tests break to the suspected change lists added since the last green test run.  I think the FreeBSD buildbot (?) does this as well, but most don't.  (Every once in a while a buildbot email goes to lldb-dev, but it's a rare event --- maybe we just never break the build :-P).</div>
<div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div><br></div><div>Those three points are a big barrier for me and may be to many others out there, to even contribute a trivial patch. Not that I have any spare time to do any real contribution, but even...</div>

<div><br></div></div></blockquote><div><br></div><div>Yep - unfortunately projects like LLVM see hundreds of patches a day and changing such a system takes a long, long time, and agreement among a large set of developers across many organizations.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div></div><div>You may be aware of those issues already of course.</div>
<div><br></div><div>Thanks for reading.</div><div><br></div></div></blockquote><div>My pleasure!  Happy to see your interest in getting involved :-)</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<div dir="ltr"><div></div><div>Best regards,<span class=""><font color="#888888"><br clear="all"><div><br></div>-- <br><div dir="ltr">Thiago Farina</div>

</font></span></div></div>
</blockquote></div><div class="gmail_extra"><br></div><div class="gmail_extra">Thanks for your comments, Thiago.  Let me know if you have any more questions.</div><div class="gmail_extra"><br></div><div class="gmail_extra">
-Todd</div>
</div></div></div>