[PATCH] D28493: [zorg] Add Polly builder for ARM target
Renato Golin via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jan 11 01:19:36 PST 2017
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
More information about the llvm-commits
mailing list