[PATCH] D28493: [zorg] Add Polly builder for ARM target

[Renato Golin via Phabricator via llvm-commits]

rengolin added inline comments.


================
Comment at: buildbot/osuosl/master/config/status.py:174
+            sendToInterestedUsers= False,
+            extraRecipients = ["efriedma at codeaurora.org"],
+            subject="Build %(builder)s Failure",
----------------
pzheng wrote:
> rengolin wrote:
> > You don't need to add this to new builders.
> We want to have someone monitor the builder, thus adding this extra stuff.
This should really pick up from the bot owner's email, not this hard-coded address. But that's not for this patch.


================
Comment at: zorg/buildbot/builders/PollyBuilder.py:100
+    if install:
+        f.addStep(ShellCommand(name="install",
+                               command=install_cmd,
----------------
Why do you need the install dir if you're not using it in the buildbot?


https://reviews.llvm.org/D28493