diff options
| author | thomasvl@chromium.org <thomasvl@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-02-03 20:50:07 +0000 |
|---|---|---|
| committer | thomasvl@chromium.org <thomasvl@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-02-03 20:50:07 +0000 |
| commit | c775e00efade22d264a07888b5c70a443552fd99 (patch) | |
| tree | 9b076918f743f2b2e3ddd4c3aa4b9e3d338a98a9 | |
| parent | 6cbd190f00302097d2a2a08086e196db66d7ae85 (diff) | |
| download | chromium_src-c775e00efade22d264a07888b5c70a443552fd99.zip chromium_src-c775e00efade22d264a07888b5c70a443552fd99.tar.gz chromium_src-c775e00efade22d264a07888b5c70a443552fd99.tar.bz2 | |
[Mac] use subprocess.Popen instead of os.popen2 to avoid warnings on 10.6 bots due to newer python version
BUG=27879
TEST=none
Review URL: http://codereview.chromium.org/570004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38013 0039d316-1c4b-4281-b951-d872f2087c98
| -rw-r--r-- | webkit/tools/layout_tests/webkitpy/layout_tests/layout_package/platform_utils_mac.py | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/webkit/tools/layout_tests/webkitpy/layout_tests/layout_package/platform_utils_mac.py b/webkit/tools/layout_tests/webkitpy/layout_tests/layout_package/platform_utils_mac.py index f64f793..5cec147 100644 --- a/webkit/tools/layout_tests/webkitpy/layout_tests/layout_package/platform_utils_mac.py +++ b/webkit/tools/layout_tests/webkitpy/layout_tests/layout_package/platform_utils_mac.py @@ -42,7 +42,8 @@ def platform_version(): def get_num_cores(): """Returns the number of cores on the machine. For hyperthreaded machines, this will be double the number of actual processors.""" - return int(os.popen2("sysctl -n hw.ncpu")[1].read()) + cmd = ['/usr/sbin/sysctl', '-n', 'hw.ncpu'] + return int(subprocess.Popen(cmd, stdout=subprocess.PIPE).stdout.read()) def baseline_path(platform=None): |