utils.py: Improve error handling of _parse_version().

This was not reporting errors if not given a string, since we
don't only pass strings parsed from project.conf but also pass
plugin defined Plugin.BST_MIN_VERSION definitions, we should
be more fault tolerant here.

1 files changed, 4 insertions, 4 deletions

diff --git a/src/buildstream/utils.py b/src/buildstream/utils.py
index 9f7690f7f..29b3bf484 100644
--- a/src/buildstream/utils.py
+++ b/src/buildstream/utils.py
@@ -1598,14 +1598,14 @@ def _is_single_threaded():
 #
 def _parse_version(version: str) -> Tuple[int, int]:
 
-    versions = version.split(".")
     try:
+        versions = version.split(".")
         major = int(versions[0])
         minor = int(versions[1])
-    except (IndexError, ValueError):
-        raise UtilError("Malformed version string: {}".format(version),)
+    except (IndexError, ValueError, AttributeError) as e:
+        raise UtilError("Malformed version string: {}".format(version),) from e
 
-    return (major, minor)
+    return major, minor
 
 
 # _get_bst_api_version():