more robust check of git version

Author: tjfulle

src/hpc_connect/util/dynamic_version.py

@@ -15,10 +15,14 @@ def version_components_from_git(full: bool = False) -> tuple[int, int, int, str]
         dir = os.path.dirname(dir)
         if dir == os.path.sep:
             raise GitRepoNotFoundError(os.path.dirname(__file__))
-    out = subprocess.check_output(
-        ["git", "-C", dir, "log", "-1", "--pretty=format:%ad %h", "--date=short"],
-        encoding="utf-8",
-    )
+    try:
+        out = subprocess.check_output(
+            ["git", "-C", dir, "log", "-1", "--pretty=format:%ad %h", "--date=short"],
+            encoding="utf-8",
+        )
+    except Exception as e:
+        raise CannotDetermineVersionFromGitError(e.args[0]) from None
+
     parts = out.split()
     date, local = parts[:2]
     major, minor, micro = [int(_) for _ in date.split("-")]
@@ -34,6 +38,10 @@ class GitRepoNotFoundError(Exception):
     pass
 
 
+class CannotDetermineVersionFromGitError(Exception):
+    pass
+
+
 def write_version_file(file: TextIO, major: int, minor: int, micro: int, local: str) -> None:
     file.write(
         f"""# Version file automatically generated by hpc_connect
@@ -43,8 +51,9 @@ def write_version_file(file: TextIO, major: int, minor: int, micro: int, local:
 
 def __getattr__(name):
 
-    from hpc_connect.util.dynamic_version import version_components_from_git
+    from hpc_connect.util.dynamic_version import CannotDetermineVersionFromGitError
     from hpc_connect.util.dynamic_version import GitRepoNotFoundError
+    from hpc_connect.util.dynamic_version import version_components_from_git
 
     if name not in ("version", "__version__", "version_info", "__version_info__"):
         raise AttributeError(name)
@@ -54,7 +63,7 @@ def __getattr__(name):
             return f"{{major}}.{{minor}}.{{micro}}+{{local}}"
         else:
             return (major, minor, micro, local)
-    except GitRepoNotFoundError:
+    except (GitRepoNotFoundError, CannotDetermineVersionFromGitError):
         if name in ("version", "__version__"):
             return __static_version__
         else: