[PATCH] D11445: [zorg] Add lldb build factory that launches scripts on slave machine
Chaoren Lin
chaorenl at google.com
Fri Jul 24 11:56:45 PDT 2015
chaoren added inline comments.
================
Comment at: zorg/buildbot/builders/LLDBBuilder.py:926
@@ +925,3 @@
+
+ # Get lldb-server binaries
+ if not buildAndroid:
----------------
chying wrote:
> chaoren wrote:
> > We don't need this until we start testing. Could you move this between build and test?
> I was thinking if this revision of binaries doesn't exist, then the build will terminate from here.
> This might happen when the build is not triggered by buildserver, like when user click Force Build from webpage.
> like when user click Force Build from webpage
In that case shouldn't we just fetch the most recent lldb-server?
================
Comment at: zorg/buildbot/builders/LLDBBuilder.py:943
@@ +942,3 @@
+
+ # Configure
+ getShellCommandStep(f, name='cmake local',
----------------
chying wrote:
> chaoren wrote:
> > We can probably condense cleanup, checkout, and configure into one configure script. I don't think the first two steps would be useful to see separately.
> We had some build failures with source checkout due to SVN server issue.
> How about combine cleanup and checkout?
Sounds good.
http://reviews.llvm.org/D11445
More information about the llvm-commits
mailing list