]> git.ipfire.org Git - thirdparty/mkosi.git/commitdiff
Simplify finalize_firmware_variables() a little
authorDaan De Meyer <daan.j.demeyer@gmail.com>
Wed, 22 Jan 2025 15:54:28 +0000 (16:54 +0100)
committerDaan De Meyer <daan.j.demeyer@gmail.com>
Wed, 22 Jan 2025 15:54:28 +0000 (16:54 +0100)
mkosi/qemu.py

index ae959cd9e05ac22465ae6c6a4cae6b537f9fe282..2e54b8c85d70804a8d935f0857fefbe73031bac7 100644 (file)
@@ -717,7 +717,7 @@ def finalize_firmware_variables(
     ovmf: OvmfConfig,
     stack: contextlib.ExitStack,
 ) -> tuple[Path, str]:
-    ovmf_vars = stack.enter_context(tempfile.NamedTemporaryFile(prefix="mkosi-ovmf-vars-"))
+    ovmf_vars = Path(stack.enter_context(tempfile.NamedTemporaryFile(prefix="mkosi-ovmf-vars-")).name)
     if config.firmware_variables in (None, Path("custom"), Path("microsoft")):
         ovmf_vars_format = ovmf.vars_format
     else:
@@ -729,7 +729,7 @@ def finalize_firmware_variables(
             [
                 "virt-fw-vars",
                 "--input", workdir(ovmf.vars),
-                "--output", workdir(Path(ovmf_vars.name)),
+                "--output", workdir(ovmf_vars),
                 "--enroll-cert", workdir(config.secure_boot_certificate),
                 "--add-db", "OvmfEnrollDefaultKeys", workdir(config.secure_boot_certificate),
                 "--no-microsoft",
@@ -738,7 +738,7 @@ def finalize_firmware_variables(
             ],
             sandbox=config.sandbox(
                 options=[
-                    "--bind", ovmf_vars.name, workdir(Path(ovmf_vars.name)),
+                    "--bind", ovmf_vars, workdir(ovmf_vars),
                     "--ro-bind", ovmf.vars, workdir(ovmf.vars),
                     "--ro-bind", config.secure_boot_certificate, workdir(config.secure_boot_certificate),
                 ],
@@ -750,9 +750,9 @@ def finalize_firmware_variables(
             if config.firmware_variables == Path("microsoft") or not config.firmware_variables
             else config.firmware_variables
         )
-        shutil.copy2(vars, Path(ovmf_vars.name))
+        shutil.copy2(vars, ovmf_vars)
 
-    return Path(ovmf_vars.name), ovmf_vars_format
+    return ovmf_vars, ovmf_vars_format
 
 
 def apply_runtime_size(config: Config, image: Path) -> None: