[PATCH] D29050: [zorg] Add AOSP builder

[Tobias Grosser via Phabricator via llvm-commits]

grosser added a comment.

Thanks for explaining how to setup the buildslave. Could you add the information (the link what to install, the need for java, the information which AOSP repo to checkout, ...) as comment to the AOSPBuilder? Also, if the AOSP checkout is just a one-time thing, this could still be part of the builder, as long as we make sure we do not clean it in each build, but just update it.

Best,
Tobias



================
Comment at: zorg/buildbot/builders/AOSPBuilder.py:84
+                           description=["install llvm"],
+                           workdir=llvm_objdir))
+
----------------
Why do you need to list these steps again? Can you not just use include the normal Polly bullder (that you recently extended to allow the installation of clang)?


https://reviews.llvm.org/D29050