Superficial gyp changes to not immediately error out on Windows

Superficial gyp changes to not immediately error out on Windows

Just guarding the usage of $SDKROOT so that ninja doesn't error out
on invalid $ escape on parsing.

R=mark@chromium.org
BUG=crashpad:1
Committed: https://chromium.googlesource.com/crashpad/crashpad/+/7dda7b3228817cfad06e6ee523271c24ddabe416

[scottmg, 2014-09-30 19:52:30]

scottmg@chromium.org changed reviewers:
+ mark@chromium.org

[scottmg, 2014-09-30 19:53:04]

(let me know if you'd rather not deal with these yet, was just planning on
getting things more or less compiling so I can start investigating other stuff
on a Windows host)

[Mark Mentovai, 2014-09-30 19:55:01]

Thanks, Scott, I’m happy to deal with this now.

https://codereview.chromium.org/620553004/diff/1/tools/tools.gyp
File tools/tools.gyp (right):

https://codereview.chromium.org/620553004/diff/1/tools/tools.gyp#newcode75
tools/tools.gyp:75: 'conditions': [
None of these tools make any sense off of Mac. We should just move all three
tools into a Mac-only condition.

[scottmg, 2014-09-30 19:59:50]

Thanks

https://codereview.chromium.org/620553004/diff/1/tools/tools.gyp
File tools/tools.gyp (right):

https://codereview.chromium.org/620553004/diff/1/tools/tools.gyp#newcode75
tools/tools.gyp:75: 'conditions': [
On 2014/09/30 19:55:01, Mark Mentovai wrote:
> None of these tools make any sense off of Mac. We should just move all three
> tools into a Mac-only condition.

Done.

[Mark Mentovai, 2014-09-30 20:09:43]

LGTM. I’ve added you to crashpad-committers. Thank you!

[scottmg, 2014-09-30 20:11:07]

Committed patchset #2 (id:20001) manually as
7dda7b3228817cfad06e6ee523271c24ddabe416 (presubmit successful).

[scottmg, 2014-09-30 20:11:15]

Thanks!