[PATCH] [zorg] Fix get slave environment in LLDB Windows builder.

[Rick Foos]

Something like that.

The os.path.join's run on the linux master, so they are forward slash. I 
changed that part for double backslash.

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


On 01/23/2015 09:43 AM, Zachary Turner wrote:
> Strange. It's resolving the environment var, just not finding anything 
> there. I wonder if its an issue wit / and \. Can you try 
> os.path.normpath() first?
> On Thu, Jan 22, 2015 at 10:26 PM Rick Foos <rfoos at codeaurora.org 
> <mailto:rfoos at codeaurora.org>> wrote:
>
>     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,
>     operable program or batch file.
>     program finished with exit code 1
>     elapsedTime=0.109000
>
>
>     REPOSITORY
>       rL LLVM
>
>     http://reviews.llvm.org/D7017
>
>     EMAIL PREFERENCES
>     http://reviews.llvm.org/settings/panel/emailpreferences/
>
>
>
>     _______________________________________________
>     llvm-commits mailing list
>     llvm-commits at cs.uiuc.edu <mailto:llvm-commits at cs.uiuc.edu>
>     http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>

-- 
Rick Foos
Qualcomm Innovation Center, Inc.
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150123/d387b5c5/attachment.html>