[PATCH] [zorg] add llgo builder, osuosl config

[Andrew Wilkins]

In http://reviews.llvm.org/D7989#138562, @gkistanova wrote:

> Thanks for working on this, Andrew!
>
> You may want to consider clean builds as well as incremental builds supported now.
>  In this case, you would need to support at least a 'clean' property and add a step to remove the llvm_objdir with all the content if the property is set (by using the doStepIf param). This is not a show stopper for this patch, though.


Thanks for the review, and suggestions. I've added the clean param, and defaulted it to True as in LLVMBuilder and ClangBuilder. Also removed some unnecessary imports.


================
Comment at: zorg/buildbot/builders/LLGoBuilder.py:24
@@ +23,3 @@
+    f.addStep(buildbot.steps.shell.SetProperty(name="get_builddir",
+                                               command=["pwd"],
+                                               property="builddir",
----------------
gkistanova wrote:
> You import from buildbot.steps.shell, why not import SetProperty there as well?
> 
Done. (I had copied another builder.)

================
Comment at: zorg/buildbot/builders/LLGoBuilder.py:65
@@ +64,3 @@
+                               command=["ninja", "llgo"],
+                               haltOnFailure=True,
+                               description=["build llgo"],
----------------
gkistanova wrote:
> Could you use NinjaCommand instead of direct invoking ShellCommand with "ninja" here and below, please?
> 
Done.

http://reviews.llvm.org/D7989

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/