Fixes original get visual studio environment from running on master to run on buildslave.
get LLDB source is now a function.
Default clean step set to false.
Adds install step, and test (ignore errors) step.
Differential D7017
[zorg] Fix get slave environment in LLDB Windows builder. rfoos on Jan 16 2015, 11:19 AM. Authored by
Details
Fixes original get visual studio environment from running on master to run on buildslave. get LLDB source is now a function. Default clean step set to false. Adds install step, and test (ignore errors) step.
Diff Detail
Event Timeline
Comment Actions Duplicate imports removed. Drop in replacement for existing builders. Comment Actions Hi Rick, Please see my comment in-line. Also, could you re-generate the patch, please? Thanks
Comment Actions Sorry about RemoveDirectory. It was part of some internal modifications. I change clean to the RemoveDirectory step I'm going to add in the Thanks Comment Actions Hi Zachary, The nifty VS120COMNTOOLS environment variable doesn't work on your Windows 8. I have no idea why, but maybe you need a filepath like you were using before. So much for a default that runs on any windows machine :( The slave_envCmd argument will let you pass something different. Rick '\"C:\Program Files (x86)\Microsoft Visual Studio 12.0\Common7\Tools\/../../VC/vcvarsall.bat\"' is not recognized as an internal or external command, Comment Actions Strange. It's resolving the environment var, just not finding anything Comment Actions Something like that. The os.path.join's run on the linux master, so they are forward slash. I It looks like the \"'s are not being escaped, which is still a puzzle. I've got a patch that changes everything that wasn't working right. A few rounds and we'll have this resolved. Rick |
Duplicate line removed.