From 1faa0a993aa989efbc7044f8eb39b7728d82d369 Mon Sep 17 00:00:00 2001
From: Adam Turner <9087854+aa-turner@users.noreply.github.com>
Date: Tue, 15 Apr 2025 20:43:36 +0100
Subject: [PATCH] Create a function for ``chmod`` operations

---
 build_docs.py | 33 +++++++++++++++------------------
 1 file changed, 15 insertions(+), 18 deletions(-)

diff --git a/build_docs.py b/build_docs.py
index 2f164f8..d1afdf2 100755
--- a/build_docs.py
+++ b/build_docs.py
@@ -54,6 +54,7 @@
 import re
 import shlex
 import shutil
+import stat
 import subprocess
 import sys
 import venv
@@ -744,18 +745,7 @@ def copy_build_to_webroot(self, http: urllib3.PoolManager) -> None:
                 group=self.group,
                 recursive=True,
             )
-            run(["chmod", "-R", "o+r", self.checkout / "Doc" / "build" / "html"])
-            run([
-                "find",
-                self.checkout / "Doc" / "build" / "html",
-                "-type",
-                "d",
-                "-exec",
-                "chmod",
-                "o+x",
-                "{}",
-                ";",
-            ])
+            chmod_make_readable(self.checkout / "Doc" / "build" / "html")
             run([
                 "rsync",
                 "-a",
@@ -770,12 +760,7 @@ def copy_build_to_webroot(self, http: urllib3.PoolManager) -> None:
             # Copy archive files to /archives/
             logging.debug("Copying dist files.")
             chgrp(self.checkout / "Doc" / "dist", group=self.group, recursive=True)
-            run([
-                "chmod",
-                "-R",
-                "o+r",
-                self.checkout / "Doc" / "dist",
-            ])
+            chmod_make_readable(self.checkout / "Doc" / "dist")
             run(["mkdir", "-m", "o+rx", "-p", target / "archives"])
             chgrp(target / "archives", group=self.group)
             run([
@@ -907,6 +892,18 @@ def chgrp(
         logging.warning("Can't change group of %s: %s", path, str(err))
 
 
+def chmod_make_readable(path: Path, /, mode: int = stat.S_IROTH) -> None:
+    if not path.is_dir():
+        raise ValueError
+
+    path.chmod(path.stat().st_mode | stat.S_IROTH | stat.S_IXOTH)  # o+rx
+    for p in path.rglob("*"):
+        if p.is_dir():
+            p.chmod(p.stat().st_mode | stat.S_IROTH | stat.S_IXOTH)  # o+rx
+        else:
+            p.chmod(p.stat().st_mode | stat.S_IROTH)  # o+r
+
+
 def format_seconds(seconds: float) -> str:
     hours, remainder = divmod(seconds, 3600)
     minutes, seconds = divmod(remainder, 60)