commit 5b9f90e5926632f0892e7ca6e51c916c7a1fb5f0
author viettrungluu@chromium.org <viettrungluu@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> Wed Mar 05 03:28:24 2014 +0000
committer viettrungluu@chromium.org <viettrungluu@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> Wed Mar 05 03:28:24 2014 +0000
tree f36ab4bb4e0ee648719bff0100f8619ece89aa5d
parent f634964529d7c6014b8758814651fe0ff78d5c74

Make toolchain2013.py work properly if there's a space in the temp path.

TBR=maruel@chromium.org
R=scottmg@chromium.org

Review URL: https://codereview.chromium.org/184103027

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@254932 0039d316-1c4b-4281-b951-d872f2087c98

win_toolchain/toolchain2013.py

diff --git a/win_toolchain/toolchain2013.py b/win_toolchain/toolchain2013.py
index 0cda590..201e6c7 100755
--- a/win_toolchain/toolchain2013.py
+++ b/win_toolchain/toolchain2013.py
@@ -196,10 +196,11 @@
       'Running sdksetup.exe to download Win8 SDK (may request elevation)...\n')
   count = 0
   while count < 5:
-    rc = os.system(target_path + ' /quiet '
-                   '/features OptionId.WindowsDesktopDebuggers '
-                             'OptionId.WindowsDesktopSoftwareDevelopmentKit '
-                   '/layout ' + standalone_path)
+    rc = subprocess.call([target_path,
+                          '/quiet',
+                          '/features', 'OptionId.WindowsDesktopDebuggers',
+                              'OptionId.WindowsDesktopSoftwareDevelopmentKit',
+                          '/layout', standalone_path])
     if rc == 0:
       return standalone_path
     count += 1