<div dir="ltr">I'd probably avoid introducing a different ordering of the name "checkout_clang_extra_tools" (where the repository is clang_tools_extra, etc - so just name it "checkout_clang_tools_extra" for consistency)<br>
<br>The stage two part could maybe benefit from an early return rather than indenting the whole thing<br><br>Other than that it looks like a totally plausible/good start to me.</div><div class="gmail_extra"><br><br><div class="gmail_quote">
On Thu, Sep 4, 2014 at 8:13 AM, Renato Golin <span dir="ltr"><<a href="mailto:renato.golin@linaro.org" target="_blank">renato.golin@linaro.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi,<br>
<br>
I'm trying to move from autoconf buildbots to CMake+Ninja on the ARM<br>
bots, so that I can also test compiler-rt and others. Attached is my<br>
new ClangBuilder function to do that.<br>
<br>
It handles clean|unclean builds, multiple stages, testing<br>
stage1|stage2|stage1+2, clang-tools-extra, compiler-rt, etc.<br>
<br>
I have four combinations of bots {clean, unclean} x {check1,<br>
self-host+check2} and it's working great. Please let me know how can I<br>
improve, or if I can just commit and continue updating it as I<br>
progress.<br>
<br>
I'd like to use it as soon as possible to make my private bots public.<br>
<br>
cheers,<br>
--renato<br>
<br>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div><br></div>