diff options
| author | dsinclair <dsinclair@chromium.org> | 2016-04-28 06:17:40 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2016-04-28 06:17:40 -0700 |
| commit | 3b5cb78353f92ad7d23ace4e452ed26acf9aeaa7 (patch) | |
| tree | dd0c2972880da75840a94c9dbe538607aaa10c05 /testing/tools/common.py | |
| parent | bb0d446df18ee34504a165f3fc96fbb81b274f31 (diff) | |
| download | pdfium-3b5cb78353f92ad7d23ace4e452ed26acf9aeaa7.tar.xz | |
Remove output redirect from corpus test runner.
The corpus tests redirect output to a string but they just output it
later anyway. This CL removes the output redirect and removes the
redirect_output flag which will always be False now.
Review-Url: https://codereview.chromium.org/1927633002
Diffstat (limited to 'testing/tools/common.py')
| -rwxr-xr-x | testing/tools/common.py | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/testing/tools/common.py b/testing/tools/common.py index 6e9de7c82c..1e1d257f48 100755 --- a/testing/tools/common.py +++ b/testing/tools/common.py @@ -18,12 +18,9 @@ def os_name(): raise Exception('Confused, can not determine OS, aborting.') -def RunCommand(cmd, redirect_output=False): +def RunCommand(cmd): try: - if redirect_output: - sys.stdout.write(subprocess.check_output(cmd, stderr=subprocess.STDOUT)) - else: - subprocess.check_call(cmd) + subprocess.check_call(cmd) return None except subprocess.CalledProcessError as e: return e |