commit e4cf5322c65802deb38b66e50893ae751c4e10a0
author Katherine Threlkeld <kathrelkeld@chromium.org> Wed Sep 06 16:47:59 2017 -0700
committer chrome-bot <chrome-bot@chromium.org> Thu Oct 05 18:06:45 2017 -0700
tree b5cdf2a1a5250969d6fc9144ebb2b402d3860fee
parent 804df65ba2981d81c1f5ecb9cda46b744b3d8527

[Provingground] Add check to crosdl.py

If gsutil is not installed, detect that early and bail.

TEST=correctly fails with other commands that are not insatlled
BUG=None

Change-Id: I7aa4e88a585df3b9471b4e96e1df3ce4033e53b9
Reviewed-on: https://chromium-review.googlesource.com/699050
Commit-Ready: Katherine Threlkeld <kathrelkeld@chromium.org>
Tested-by: Katherine Threlkeld <kathrelkeld@chromium.org>
Reviewed-by: Kalin Stoyanov <kalin@chromium.org>

provingground/crosdl.py

diff --git a/provingground/crosdl.py b/provingground/crosdl.py
index 99510a9..e737291 100755
--- a/provingground/crosdl.py
+++ b/provingground/crosdl.py
@@ -1,4 +1,4 @@
-#!/usr/bin/python2
+#!/usr/bin/env python2
 # Copyright (c) 2014 The Chromium OS Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
@@ -127,6 +127,13 @@
                             'harddrive).'))
   arguments = parser.parse_args()
 
+  # Ensure gsutil is installed.
+  try:
+    subprocess.check_output(['which', 'gsutil'])
+  except subprocess.CalledProcessError:
+    print ('Error finding gsutil!  Please make sure it is installed.')
+    return
+
   # Find src/ dir.
   script_file_dir = os.path.dirname(os.path.realpath(__file__))
   src_dir = os.path.join(script_file_dir, REL_SRC_DIR)