utils: run: remove redundant redirect settings
capture_output=True implies both redirect_std{out,err}=True so we
can delete the latter settings when the former is active.
Change-Id: I2ad9383b3c0ea244b8984577ca4196f5980c76a7
Bug: None
Test: unittests pass
diff --git a/pre-upload.py b/pre-upload.py
index 467e6c8..d7c12aa 100755
--- a/pre-upload.py
+++ b/pre-upload.py
@@ -419,8 +419,7 @@
a blank string upon failure.
"""
cmd = ['repo', 'forall', '.', '-c', 'echo ${REPO_PROJECT}']
- return rh.utils.run(cmd, capture_output=True, redirect_stderr=True,
- cwd=path).stdout.strip()
+ return rh.utils.run(cmd, capture_output=True, cwd=path).stdout.strip()
def direct_main(argv):
@@ -452,8 +451,7 @@
# project from CWD.
if opts.dir is None:
cmd = ['git', 'rev-parse', '--git-dir']
- git_dir = rh.utils.run(cmd, capture_output=True,
- redirect_stderr=True).stdout.strip()
+ git_dir = rh.utils.run(cmd, capture_output=True).stdout.strip()
if not git_dir:
parser.error('The current directory is not part of a git project.')
opts.dir = os.path.dirname(os.path.abspath(git_dir))