15368: Don't require git to install arvados-python-client
authorPeter Amstutz <pamstutz@veritasgenetics.com>
Fri, 14 Jun 2019 17:10:57 +0000 (13:10 -0400)
committerPeter Amstutz <pamstutz@veritasgenetics.com>
Fri, 14 Jun 2019 17:34:32 +0000 (13:34 -0400)
It was catching CalledProcessError but if git is not installed the
error is actually OSError.

Arvados-DCO-1.1-Signed-off-by: Peter Amstutz <pamstutz@veritasgenetics.com>

sdk/python/arvados_version.py

index 2e6484cabdf1e71d39f5fe21139b29c2ce09ad93..692146465216448b7cb3f95fa8ce20bf8beff83c 100644 (file)
@@ -34,7 +34,7 @@ def get_version(setup_dir, module):
     else:
         try:
             save_version(setup_dir, module, git_latest_tag() + git_timestamp_tag())
-        except subprocess.CalledProcessError:
+        except (subprocess.CalledProcessError, OSError):
             pass
 
     return read_version(setup_dir, module)