From 6ed42e3ca13199a423b61278f5e85921bfbf76d2 Mon Sep 17 00:00:00 2001 From: 0xdeadd Date: Wed, 13 May 2026 11:36:42 -0400 Subject: [PATCH] fix(efi): collapse fmask/dmask dedup to dict.fromkeys one-liner Per @Torxed's review feedback. Same semantics as the previous loop (dedupe by exact-string match) but shorter. dict.fromkeys preserves insertion order, where set() would not. --- archinstall/lib/installer.py | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/archinstall/lib/installer.py b/archinstall/lib/installer.py index 676bc0fd..a05acd9f 100644 --- a/archinstall/lib/installer.py +++ b/archinstall/lib/installer.py @@ -378,9 +378,7 @@ class Installer: options = list(part_mod.mount_options) if part_mod.is_efi(): - for opt in ('fmask=0077', 'dmask=0077'): - if opt not in options: - options.append(opt) + options = list(dict.fromkeys(options + ['fmask=0077', 'dmask=0077'])) mount(part_mod.dev_path, target, options=options) elif part_mod.fs_type == FilesystemType.BTRFS: