treewide: exchange str for os.fspath were appropriate

All objects have a __str__ method, but pathlike objects define a protocol
__fspath__, let's use that where we haven't so far.

diff --git a/mkosi/__init__.py b/mkosi/__init__.py
index 02aa0a734766070e7efeecf5568a8fde5b19dbfa..f25523d4912b7d7a8a8827776cf44287aec7de3d 100644 (file)
--- a/mkosi/__init__.py
+++ b/mkosi/__init__.py
@@ -508,7 +508,7 @@ def finalize_scripts(config: Config, scripts: Mapping[str, Sequence[PathString]]
                         )
                     )
 
-                f.write(f'exec {shlex.join(str(s) for s in script)} "$@"\n')
+                f.write(f'exec {shlex.join(os.fspath(s) for s in script)} "$@"\n')
 
             make_executable(Path(d) / name)
             os.utime(Path(d) / name, (0, 0))
@@ -834,7 +834,7 @@ def run_build_scripts(context: Context) -> None:
                     "--bind", context.artifacts, "/work/artifacts",
                     "--bind", context.package_dir, "/work/packages",
                     *(
-                        ["--bind", str(context.config.build_dir), "/work/build"]
+                        ["--bind", os.fspath(context.config.build_dir), "/work/build"]
                         if context.config.build_dir
                         else []
                     ),
@@ -904,7 +904,7 @@ def run_postinst_scripts(context: Context) -> None:
                     "--bind", context.artifacts, "/work/artifacts",
                     "--bind", context.package_dir, "/work/packages",
                     *(
-                        ["--ro-bind", str(context.config.build_dir), "/work/build"]
+                        ["--ro-bind", os.fspath(context.config.build_dir), "/work/build"]
                         if context.config.build_dir
                         else []
                     ),
@@ -973,7 +973,7 @@ def run_finalize_scripts(context: Context) -> None:
                     "--bind", context.artifacts, "/work/artifacts",
                     "--bind", context.package_dir, "/work/packages",
                     *(
-                        ["--ro-bind", str(context.config.build_dir), "/work/build"]
+                        ["--ro-bind", os.fspath(context.config.build_dir), "/work/build"]
                         if context.config.build_dir
                         else []
                     ),
@@ -1290,10 +1290,10 @@ def finalize_default_initrd(
         "--compress-level", str(config.compress_level),
         "--with-network", str(config.with_network),
         "--cache-only", str(config.cacheonly),
-        *(["--output-directory", str(output_dir)] if output_dir else []),
-        *(["--workspace-directory", str(config.workspace_dir)] if config.workspace_dir else []),
-        *(["--cache-directory", str(config.cache_dir)] if config.cache_dir else []),
-        *(["--package-cache-directory", str(config.package_cache_dir)] if config.package_cache_dir else []),
+        *(["--output-directory", os.fspath(output_dir)] if output_dir else []),
+        *(["--workspace-directory", os.fspath(config.workspace_dir)] if config.workspace_dir else []),
+        *(["--cache-directory", os.fspath(config.cache_dir)] if config.cache_dir else []),
+        *(["--package-cache-directory", os.fspath(config.package_cache_dir)] if config.package_cache_dir else []),  # noqa: E501
         *(["--local-mirror", str(config.local_mirror)] if config.local_mirror else []),
         "--incremental", str(config.incremental),
         *(f"--package={package}" for package in config.initrd_packages),
@@ -1315,14 +1315,14 @@ def finalize_default_initrd(
         *(["--hostname", config.hostname] if config.hostname else []),
         *(["--root-password", rootpwopt] if rootpwopt else []),
         *([f"--environment={k}='{v}'" for k, v in config.environment.items()]),
-        *(["--tools-tree", str(config.tools_tree)] if config.tools_tree and tools else []),
+        *(["--tools-tree", os.fspath(config.tools_tree)] if config.tools_tree and tools else []),
         "--tools-tree-certificates", str(config.tools_tree_certificates),
-        *([f"--extra-search-path={p}" for p in config.extra_search_paths]),
+        *([f"--extra-search-path={os.fspath(p)}" for p in config.extra_search_paths]),
         *(["--proxy-url", config.proxy_url] if config.proxy_url else []),
         *([f"--proxy-exclude={host}" for host in config.proxy_exclude]),
-        *(["--proxy-peer-certificate", str(p)] if (p := config.proxy_peer_certificate) else []),
-        *(["--proxy-client-certificate", str(p)] if (p := config.proxy_client_certificate) else []),
-        *(["--proxy-client-key", str(p)] if (p := config.proxy_client_key) else []),
+        *(["--proxy-peer-certificate", os.fspath(p)] if (p := config.proxy_peer_certificate) else []),
+        *(["--proxy-client-certificate", os.fspath(p)] if (p := config.proxy_client_certificate) else []),
+        *(["--proxy-client-key", os.fspath(p)] if (p := config.proxy_client_key) else []),
         "--selinux-relabel", str(relabel),
         "--include=mkosi-initrd",
     ]  # fmt: skip
@@ -4332,21 +4332,21 @@ def finalize_default_tools(config: Config, *, resources: Path) -> Config:
         "--repository-key-check", str(config.repository_key_check),
         "--repository-key-fetch", str(config.repository_key_fetch),
         "--cache-only", str(config.cacheonly),
-        *(["--output-directory", str(config.output_dir)] if config.output_dir else []),
-        *(["--workspace-directory", str(config.workspace_dir)] if config.workspace_dir else []),
-        *(["--cache-directory", str(config.cache_dir)] if config.cache_dir else []),
-        *(["--package-cache-directory", str(config.package_cache_dir)] if config.package_cache_dir else []),
+        *(["--output-directory", os.fspath(config.output_dir)] if config.output_dir else []),
+        *(["--workspace-directory", os.fspath(config.workspace_dir)] if config.workspace_dir else []),
+        *(["--cache-directory", os.fspath(config.cache_dir)] if config.cache_dir else []),
+        *(["--package-cache-directory", os.fspath(config.package_cache_dir)] if config.package_cache_dir else []),  # noqa: E501
         "--incremental", str(config.incremental),
         *([f"--package={package}" for package in config.tools_tree_packages]),
-        *([f"--package-directory={directory}" for directory in config.tools_tree_package_directories]),
+        *([f"--package-directory={os.fspath(directory)}" for directory in config.tools_tree_package_directories]),  # noqa: E501
         "--output=tools",
         *(["--source-date-epoch", str(config.source_date_epoch)] if config.source_date_epoch is not None else []),  # noqa: E501
         *([f"--environment={k}='{v}'" for k, v in config.environment.items()]),
         *(["--proxy-url", config.proxy_url] if config.proxy_url else []),
         *([f"--proxy-exclude={host}" for host in config.proxy_exclude]),
-        *(["--proxy-peer-certificate", str(p)] if (p := config.proxy_peer_certificate) else []),
-        *(["--proxy-client-certificate", str(p)] if (p := config.proxy_client_certificate) else []),
-        *(["--proxy-client-key", str(p)] if (p := config.proxy_client_key) else []),
+        *(["--proxy-peer-certificate", os.fspath(p)] if (p := config.proxy_peer_certificate) else []),
+        *(["--proxy-client-certificate", os.fspath(p)] if (p := config.proxy_client_certificate) else []),
+        *(["--proxy-client-key", os.fspath(p)] if (p := config.proxy_client_key) else []),
     ]  # fmt: skip
 
     _, [tools] = parse_config(
@@ -4412,7 +4412,7 @@ def run_clean_scripts(config: Config) -> None:
                             "--dir", "/work/out",
                             "--ro-bind", script, "/work/clean",
                             "--ro-bind", json, "/work/config.json",
-                            *(["--bind", str(o), "/work/out"] if (o := config.output_dir_or_cwd()).exists() else []),  # noqa: E501
+                            *(["--bind", os.fspath(o), "/work/out"] if (o := config.output_dir_or_cwd()).exists() else []),  # noqa: E501
                             *sources,
                         ],
                     ),

diff --git a/mkosi/bootloader.py b/mkosi/bootloader.py
index bc09656723defa675ca159d8ef292049d98762cf..8fdd0f0f5c0be6972d12c913c994d2f2a334506e 100644 (file)
--- a/mkosi/bootloader.py
+++ b/mkosi/bootloader.py
@@ -1,5 +1,6 @@
 import itertools
 import logging
+import os
 import shutil
 import subprocess
 import sys
@@ -208,7 +209,7 @@ def grub_mkimage(
                 "--prefix", f"/{context.config.distribution.grub_prefix()}",
                 "--output", workdir(output) if output else "/grub/core.img",
                 "--format", target,
-                *(["--sbat", str(workdir(sbat))] if sbat else []),
+                *(["--sbat", os.fspath(workdir(sbat))] if sbat else []),
                 *(["--disable-shim-lock"] if context.config.shim_bootloader == ShimBootloader.none else []),
                 "cat",
                 "cmp",
@@ -237,8 +238,8 @@ def grub_mkimage(
                 options=[
                     "--bind", directory, "/grub",
                     "--ro-bind", earlyconfig.name, workdir(Path(earlyconfig.name)),
-                    *(["--bind", str(output.parent), str(workdir(output.parent))] if output else []),
-                    *(["--ro-bind", str(sbat), str(workdir(sbat))] if sbat else []),
+                    *(["--bind", os.fspath(output.parent), os.fspath(workdir(output.parent))] if output else []),  # noqa: E501
+                    *(["--ro-bind", os.fspath(sbat), os.fspath(workdir(sbat))] if sbat else []),
                 ],
             ),
         )  # fmt: skip

diff --git a/mkosi/initrd.py b/mkosi/initrd.py
index 883f4f010727361c9300bcf331ffaf2264717307..fec4475489fdbab22925185c1ef102d8f65fb648 100644 (file)
--- a/mkosi/initrd.py
+++ b/mkosi/initrd.py
@@ -129,7 +129,7 @@ def initrd_finalize(staging_dir: str, output: str, output_dir: str) -> None:
         with umask(~0o700) if os.getuid() == 0 else cast(umask, contextlib.nullcontext()):
             Path(output_dir).mkdir(parents=True, exist_ok=True)
     else:
-        output_dir = str(Path.cwd())
+        output_dir = os.fspath(Path.cwd())
 
     log_notice(f"Copying {staging_dir}/{output} to {output_dir}/{output}")
     # mkosi symlinks the expected output image, so dereference it

diff --git a/mkosi/mounts.py b/mkosi/mounts.py
index e32d4879510343d9a1a8bd5623c96fb065916a09..5416d57a00c67ed6f07657ea9d81af4fb0d474dc 100644 (file)
--- a/mkosi/mounts.py
+++ b/mkosi/mounts.py
@@ -50,7 +50,12 @@ def mount_overlay(
         )
 
         try:
-            with OverlayOperation(tuple(str(p) for p in lowerdirs), str(upperdir), str(workdir), str(dst)):
+            with OverlayOperation(
+                tuple(os.fspath(p) for p in lowerdirs),
+                os.fspath(upperdir),
+                os.fspath(workdir),
+                os.fspath(dst),
+            ):
                 yield dst
         finally:
             delete_whiteout_files(upperdir)

diff --git a/mkosi/run.py b/mkosi/run.py
index b6887972c47fdd3405d5285ac217e1361038b20b..3cc3b01ee26ca6313c7b1185e039917c2eece040 100644 (file)
--- a/mkosi/run.py
+++ b/mkosi/run.py
@@ -492,7 +492,7 @@ def nosandbox(
 
 def workdir(path: Path, sandbox: Optional[SandboxProtocol] = None) -> str:
     subdir = "/" if sandbox and sandbox == nosandbox else "/work"
-    return joinpath(subdir, str(path))
+    return joinpath(subdir, os.fspath(path))
 
 
 def finalize_passwd_symlinks(root: PathString) -> list[PathString]:
@@ -691,7 +691,7 @@ def sandbox_cmd(
                 cmdline += [
                     "--overlay-lowerdir", overlay / d,
                     "--overlay-upperdir", tmp or "tmpfs",
-                    *(["--overlay-workdir", str(work)] if work else []),
+                    *(["--overlay-workdir", os.fspath(work)] if work else []),
                     "--overlay", Path("/") / d,
                 ]  # fmt: skip
             elif not relaxed:

diff --git a/mkosi/tree.py b/mkosi/tree.py
index 89c0d447720473d83744db21025c7ac699537cca..1bc5e6d837e588471c814a91fc5596caf2279b77 100644 (file)
--- a/mkosi/tree.py
+++ b/mkosi/tree.py
@@ -19,7 +19,7 @@ from mkosi.versioncomp import GenericVersion
 
 
 def is_subvolume(path: Path) -> bool:
-    return path.is_dir() and path.stat().st_ino == 256 and statfs(str(path)) == BTRFS_SUPER_MAGIC
+    return path.is_dir() and path.stat().st_ino == 256 and statfs(os.fspath(path)) == BTRFS_SUPER_MAGIC
 
 
 def cp_version(*, sandbox: SandboxProtocol = nosandbox) -> GenericVersion:
@@ -42,7 +42,7 @@ def make_tree(
 ) -> Path:
     path = path.absolute()
 
-    if statfs(str(path.parent)) != BTRFS_SUPER_MAGIC:
+    if statfs(os.fspath(path.parent)) != BTRFS_SUPER_MAGIC:
         if use_subvolumes == ConfigFeature.enabled:
             die(f"Subvolumes requested but {path} is not located on a btrfs filesystem")
 
@@ -134,7 +134,7 @@ def copy_tree(
         use_subvolumes == ConfigFeature.disabled
         or not preserve
         or not is_subvolume(src)
-        or statfs(str(dst.parent)) != BTRFS_SUPER_MAGIC
+        or statfs(os.fspath(dst.parent)) != BTRFS_SUPER_MAGIC
         or (dst.exists() and (not dst.is_dir() or any(dst.iterdir())))
     ):
         with preserve_target_directories_stat(src, dst) if not preserve else contextlib.nullcontext():

diff --git a/tests/test_config.py b/tests/test_config.py
index 8aa92d0993bed07fbb71075d58b7a31a4ba240c9..2360e943bf871f0a2586dafedebe5839539e57dc 100644 (file)
--- a/tests/test_config.py
+++ b/tests/test_config.py
@@ -1188,7 +1188,7 @@ def test_specifiers(tmp_path: Path) -> None:
             "Image": "",
             "ImageId": "my-image-id",
             "ImageVersion": "1.2.3",
-            "OutputDirectory": str(Path.cwd() / "abcde"),
+            "OutputDirectory": os.fspath(Path.cwd() / "abcde"),
             "Output": "test",
             "ConfigRootDirectory": os.fspath(d),
             "ConfigRootConfdir": os.fspath(d),

diff --git a/tests/test_signing.py b/tests/test_signing.py
index 7b8f3e8047cc76c07bb3dcd3cf48e245dc1887c7..8c7d72a527d57efa73034e14c540258587e8ecdc 100644 (file)
--- a/tests/test_signing.py
+++ b/tests/test_signing.py
@@ -1,6 +1,7 @@
 # SPDX-License-Identifier: LGPL-2.1-or-later
 
 
+import os
 import tempfile
 from pathlib import Path
 
@@ -50,7 +51,7 @@ def test_signing_checksums_with_gpg(config: ImageConfig) -> None:
         signing_cert = tmp_path / "signing-cert.pgp"
         gnupghome = tmp_path / ".gnupg"
         gnupghome.mkdir()
-        env = dict(GNUPGHOME=str(gnupghome))
+        env = dict(GNUPGHOME=os.fspath(gnupghome))
 
         # create a brand new signing key
         run(