win toolchain: Always use ; for all env vars except PATH, even on non-Windows.

win toolchain: Always use ; for all env vars except PATH, even on non-Windows.

lld-link expects LIB to be ;-separated even on non-Windows.  Hopefully
it'll eventually use a flag for system lib dirs instead of looking at env vars,
but for now it looks at env vars.

(clang-cl also expects INCLUDE to be ;-separated, but it already uses
-imsvc instead of the INCLUDE env var to find system include dirs.)

BUG=495204
Committed: https://crrev.com/b61345c8939bfc9e23f9a2442e681b54566120ca
Cr-Commit-Position: refs/heads/master@{#392676}

[Nico, 2016-05-10 18:17:13]

Description was changed from

==========
win toolchain: Always use ; for all env vars except PATH, even on non-Windows.

BUG=495204
==========

to

==========
win toolchain: Always use ; for all env vars except PATH, even on non-Windows.

lld-link expects LIB to be ;-separated even on non-Windows.  Hopefully
it'll eventually use a flag for system lib dirs instead of looking at env vars,
but for now it looks at env vars.

(clang-cl also expects INCLUDE to be ;-separated, but it already uses
-imsvc instead of the INCLUDE env var to find system include dirs.)

BUG=495204
==========

[Nico, 2016-05-10 18:17:32]

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

[scottmg, 2016-05-10 18:48:10]

lgtm

[Nico, 2016-05-10 18:53:51]

The CQ bit was checked by thakis@chromium.org

[commit-bot: I haz the power, 2016-05-10 18:54:30]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1970443002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1970443002/20001

[commit-bot: I haz the power, 2016-05-10 19:15:52]

Description was changed from

==========
win toolchain: Always use ; for all env vars except PATH, even on non-Windows.

lld-link expects LIB to be ;-separated even on non-Windows.  Hopefully
it'll eventually use a flag for system lib dirs instead of looking at env vars,
but for now it looks at env vars.

(clang-cl also expects INCLUDE to be ;-separated, but it already uses
-imsvc instead of the INCLUDE env var to find system include dirs.)

BUG=495204
==========

to

==========
win toolchain: Always use ; for all env vars except PATH, even on non-Windows.

lld-link expects LIB to be ;-separated even on non-Windows.  Hopefully
it'll eventually use a flag for system lib dirs instead of looking at env vars,
but for now it looks at env vars.

(clang-cl also expects INCLUDE to be ;-separated, but it already uses
-imsvc instead of the INCLUDE env var to find system include dirs.)

BUG=495204
==========

[commit-bot: I haz the power, 2016-05-10 19:15:53]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-05-10 19:18:06]

Description was changed from

==========
win toolchain: Always use ; for all env vars except PATH, even on non-Windows.

lld-link expects LIB to be ;-separated even on non-Windows.  Hopefully
it'll eventually use a flag for system lib dirs instead of looking at env vars,
but for now it looks at env vars.

(clang-cl also expects INCLUDE to be ;-separated, but it already uses
-imsvc instead of the INCLUDE env var to find system include dirs.)

BUG=495204
==========

to

==========
win toolchain: Always use ; for all env vars except PATH, even on non-Windows.

lld-link expects LIB to be ;-separated even on non-Windows.  Hopefully
it'll eventually use a flag for system lib dirs instead of looking at env vars,
but for now it looks at env vars.

(clang-cl also expects INCLUDE to be ;-separated, but it already uses
-imsvc instead of the INCLUDE env var to find system include dirs.)

BUG=495204

Committed: https://crrev.com/b61345c8939bfc9e23f9a2442e681b54566120ca
Cr-Commit-Position: refs/heads/master@{#392676}
==========

[commit-bot: I haz the power, 2016-05-10 19:18:07]

Patchset 2 (id:??) landed as
https://crrev.com/b61345c8939bfc9e23f9a2442e681b54566120ca
Cr-Commit-Position: refs/heads/master@{#392676}