Use sys.exit rather than raise SystemExit

Use sys.exit rather than raise SystemExit

TBR=iannucci@chromium.org
R=iannucci@google.com

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=247469

[iannucci1, 2014-01-28 17:43:18]

kthxlgtm

https://codereview.chromium.org/148523009/diff/1/win_toolchain/toolchain2013.py
File win_toolchain/toolchain2013.py (right):

https://codereview.chromium.org/148523009/diff/1/win_toolchain/toolchain2013....
win_toolchain/toolchain2013.py:159: sys.exit("After multiple retries, couldn't
download Win8 SDK")
annoying nit: single quotes are the preference as well

https://codereview.chromium.org/148523009/diff/1/win_toolchain/toolchain2013....
win_toolchain/toolchain2013.py:372: DeleteAllTempDirs()
Oh, derp, didn't see this and didn't read TempDirs closely and I forgot that I
had this exact same comment last time. Basically, I suck :P

[scottmg, 2014-01-28 17:47:17]

https://codereview.chromium.org/148523009/diff/1/win_toolchain/toolchain2013.py
File win_toolchain/toolchain2013.py (right):

https://codereview.chromium.org/148523009/diff/1/win_toolchain/toolchain2013....
win_toolchain/toolchain2013.py:159: sys.exit("After multiple retries, couldn't
download Win8 SDK")
On 2014/01/28 17:43:18, iannucci1 wrote:
> annoying nit: single quotes are the preference as well

Done.

(Someone told me to switch because there was a ' inside to avoid escaping, but
meh.)

https://codereview.chromium.org/148523009/diff/1/win_toolchain/toolchain2013....
win_toolchain/toolchain2013.py:372: DeleteAllTempDirs()
On 2014/01/28 17:43:18, iannucci1 wrote:
> Oh, derp, didn't see this and didn't read TempDirs closely and I forgot that I
> had this exact same comment last time. Basically, I suck :P

np, thanks for reviewing.

[commit-bot: I haz the power, 2014-01-28 17:47:32]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/scottmg@chromium.org/148523009/20001

[commit-bot: I haz the power, 2014-01-28 17:49:13]

Presubmit check for 148523009-20001 failed and returned exit status 1.

Running presubmit commit checks ...
Checking out rietveld...
Running save-description-on-failure.sh
Running push-basic.sh
Running upstream.sh
Running submit-from-new-dir.sh
Running abandon.sh
Running submodule-merge-test.sh
Running upload-local-tracking-branch.sh
Running hooks.sh
Running post-dcommit-hook-test.sh
Running upload-stale.sh
Running patch.sh
Running basic.sh

** Presubmit ERRORS **
Missing LGTM from an OWNER for these files:
    depot_tools/win_toolchain/toolchain2013.py

Presubmit checks took 98.2s to calculate.

[scottmg, 2014-01-28 17:59:43]

On 2014/01/28 17:49:13, I haz the power (commit-bot) wrote:
> Presubmit check for 148523009-20001 failed and returned exit status 1.
> 
> Running presubmit commit checks ...
> Checking out rietveld...
> Running save-description-on-failure.sh
> Running push-basic.sh
> Running upstream.sh
> Running submit-from-new-dir.sh
> Running abandon.sh
> Running submodule-merge-test.sh
> Running upload-local-tracking-branch.sh
> Running hooks.sh
> Running post-dcommit-hook-test.sh
> Running upload-stale.sh
> Running patch.sh
> Running basic.sh
> 
> ** Presubmit ERRORS **
> Missing LGTM from an OWNER for these files:
>     depot_tools/win_toolchain/toolchain2013.py
> 
> Presubmit checks took 98.2s to calculate.

Apparently I need the other iannucci. :P

[commit-bot: I haz the power, 2014-01-28 18:08:26]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/scottmg@chromium.org/148523009/20001

[commit-bot: I haz the power, 2014-01-28 18:10:07]

Change committed as 247469