Make setup_toolchain.py run on non-Windows.

Make setup_toolchain.py run on non-Windows.

Also delete an unused, commented-out block of code.

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

[Nico, 2016-03-21 01:25:45]

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

[scottmg, 2016-03-21 17:30:14]

lgtm

https://codereview.chromium.org/1822633002/diff/1/build/toolchain/win/setup_t...
File build/toolchain/win/setup_toolchain.py (right):

https://codereview.chromium.org/1822633002/diff/1/build/toolchain/win/setup_t...
build/toolchain/win/setup_toolchain.py:190: #if win_sdk_path:
I'm not sure why this is here, probably just nuke it.

[Nico, 2016-03-21 17:33:54]

Thanks!

https://codereview.chromium.org/1822633002/diff/1/build/toolchain/win/setup_t...
File build/toolchain/win/setup_toolchain.py (right):

https://codereview.chromium.org/1822633002/diff/1/build/toolchain/win/setup_t...
build/toolchain/win/setup_toolchain.py:190: #if win_sdk_path:
On 2016/03/21 17:30:14, scottmg wrote:
> I'm not sure why this is here, probably just nuke it.

I asked here:
https://codereview.chromium.org/1783773004/diff/1/build/toolchain/win/setup_t...

But deleting it in this CL sounds good to me, done.

[Nico, 2016-03-21 17:36:00]

The CQ bit was checked by thakis@chromium.org

[Nico, 2016-03-21 17:36:00]

The patchset sent to the CQ was uploaded after l-g-t-m from scottmg@chromium.org
Link to the patchset: https://codereview.chromium.org/1822633002/#ps20001
(title: "delete unused block")

[commit-bot: I haz the power, 2016-03-21 17:36:29]

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

[Nico, 2016-03-21 17:36:37]

Description was changed from

==========
Make setup_toolchain.py run on non-Windows.

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

to

==========
Make setup_toolchain.py run on non-Windows.

Also delete an unused, commented-out block of code.

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

[commit-bot: I haz the power, 2016-03-21 18:20:37]

Description was changed from

==========
Make setup_toolchain.py run on non-Windows.

Also delete an unused, commented-out block of code.

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

to

==========
Make setup_toolchain.py run on non-Windows.

Also delete an unused, commented-out block of code.

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

[commit-bot: I haz the power, 2016-03-21 18:20:38]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-03-21 18:22:02]

Description was changed from

==========
Make setup_toolchain.py run on non-Windows.

Also delete an unused, commented-out block of code.

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

to

==========
Make setup_toolchain.py run on non-Windows.

Also delete an unused, commented-out block of code.

BUG=495204

Committed: https://crrev.com/0917363388c80cb57ee5ceba36dcd047b4bf1d19
Cr-Commit-Position: refs/heads/master@{#382330}
==========

[commit-bot: I haz the power, 2016-03-21 18:22:02]

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