Merge pull request #63 from plone/fix_62

davisagli · web-flow · commit 30e557d822c3 · 2025-02-04T10:38:03.000-08:00
Fixes #62
diff --git a/cookieplone/utils/commands/__init__.py b/cookieplone/utils/commands/__init__.py
@@ -16,7 +16,7 @@ def _get_command_version(cmd: str) -> str:
             .stdout.decode()
             .strip()
         )
-    except FileNotFoundError:
+    except (FileNotFoundError, PermissionError):
         raw_version = ""
     return raw_version
 
diff --git a/news/62.bugfix b/news/62.bugfix
@@ -0,0 +1 @@
+Added an exception for sanity check command failure due to permission error. @ewohnlich

Original file line number	Diff line number	Diff line change
`@@ -16,7 +16,7 @@ def _get_command_version(cmd: str) -> str:`
`16`	`16`	`.stdout.decode()`
`17`	`17`	`.strip()`
`18`	`18`	`)`
`19`		`- except FileNotFoundError:`
	`19`	`+ except (FileNotFoundError, PermissionError):`
`20`	`20`	`raw_version = ""`
`21`	`21`	`return raw_version`
`22`	`22`
Original file line number	Diff line number	Diff line change
`@@ -0,0 +1 @@`
	`1`	`+Added an exception for sanity check command failure due to permission error. @ewohnlich`