Make CDROM an upper case option

It's an abbrevation so let's make it upper case similar to TPM and KVM.
systemd · Jan 2, 2025 · 5342480 · 5342480
1 parent 069e299
commit 5342480
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 2 deletions.
diff --git a/mkosi/config.py b/mkosi/config.py
@@ -3622,6 +3622,7 @@ def parse_ini(path: Path, only_sections: Collection[str] = ()) -> Iterator[tuple
     ),
     ConfigSetting(
         dest="cdrom",
+        name="CDROM",
         metavar="BOOLEAN",
         nargs="?",
         section="Runtime",

diff --git a/mkosi/resources/man/mkosi.1.md b/mkosi/resources/man/mkosi.1.md
@@ -1611,7 +1611,7 @@ boolean argument: either `1`, `yes`, or `true` to enable, or `0`, `no`,
 :   Configure whether to use a virtual TPM when booting a virtual machine.
     Takes a boolean value or `auto`. Defaults to `auto`.
 
-`Cdrom=`, `--cdrom=`
+`CDROM=`, `--cdrom=`
 :   Configures whether to attach the image as a CD-ROM device when booting a
     virtual machine. Takes a boolean. Defaults to `no`.
 

diff --git a/tests/test_json.py b/tests/test_json.py
@@ -118,10 +118,10 @@ def test_config() -> None:
                 }
             ],
             "BuildSourcesEphemeral": true,
+            "CDROM": false,
             "CPUs": 2,
             "CacheDirectory": "/is/this/the/cachedir",
             "CacheOnly": "always",
-            "Cdrom": false,
             "Checksum": false,
             "CleanPackageMetadata": "auto",
             "CleanScripts": [