Wrap metadata check in try-except

Not sure if this is a good idea, but here is a proposed change
2025-12-07 01:20:16 +00:00 · 2022-04-27 15:37:59 +01:00
1 changed files with 10 additions and 4 deletions
--- a/synapse/util/check_dependencies.py
+++ b/synapse/util/check_dependencies.py
@@ -54,10 +54,16 @@ class DependencyException(Exception):


 DEV_EXTRAS = {"lint", "mypy", "test", "dev"}
-RUNTIME_EXTRAS = (
-    set(metadata.metadata(DISTRIBUTION_NAME).get_all("Provides-Extra")) - DEV_EXTRAS
-)
-VERSION = metadata.version(DISTRIBUTION_NAME)
+try:
+    RUNTIME_EXTRAS = (
+        set(metadata.metadata(DISTRIBUTION_NAME).get_all("Provides-Extra")) - DEV_EXTRAS
+    )
+    VERSION = metadata.version(DISTRIBUTION_NAME)
+except Exception as e:
+    raise RuntimeError(
+        "Unable to read Synapse's package installation metadata. "
+        "This may be a problem with how Synapse has been packaged."
+    ) from e


 def _is_dev_dependency(req: Requirement) -> bool: