Commit 2241db8a authored by Bruce Dawson's avatar Bruce Dawson Committed by LUCI CQ

Avoid capture_output to support Python 3.6

autoninja.py uses subprocess.run which requires Python 3 and used
capture_output which requires Python 3.6. One user reported this as an
issue and it turns out that it is easy to avoid by using subprocess.NULL
which then means that Python versions back to 3.3 are supported.

Bug: 868590, b/174673874
Change-Id: Ife5e186d9c54747d35ff989dc2afadba5b9a57f5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2630525
Auto-Submit: Bruce Dawson <brucedawson@chromium.org>
Reviewed-by: 's avatarJustin Cohen <justincohen@chromium.org>
Commit-Queue: Bruce Dawson <brucedawson@chromium.org>
parent b7ddc5a0
......@@ -112,7 +112,9 @@ if offline or goma_disabled_env in ['true', 't', 'yes', 'y', '1']:
if use_remote_build and remote_build_is_goma:
# Check to make sure that goma is running. If not, don't start the build.
gomacc_path = os.path.join(sys.path[0], '.cipd_bin', 'gomacc')
status = subprocess.run([gomacc_path, 'port'], capture_output=True).returncode
status = subprocess.run([gomacc_path, 'port'],
stdout=subprocess.DEVNULL,
stderr=subprocess.DEVNULL).returncode
if status == 1:
print('echo Goma is not running. Use "goma_ctl start" to start it.')
sys.exit(0)
......@@ -178,4 +180,3 @@ if offline and not sys.platform.startswith('win'):
print('RBE_remote_disabled=1 GOMA_DISABLED=1 ' + ' '.join(args))
else:
print(' '.join(args))
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment