From 7f3bfd4bcccf4aa3b0f1092af4c2b69367fc17bd Mon Sep 17 00:00:00 2001 From: Skyler Grey Date: Mon, 5 Jan 2026 11:07:01 +0000 Subject: [PATCH] fix(pm): don't fail evaluation on multiple homes Change-Id: klmklpzwxtxukssooyuumxywqsruqpsw As a regression from some of the assertion refactoring... When you have multiple homes, we were previously comparing the entire config of each of them to all of the rest. This was a Bad Thing as it caused us to evaluate the whole of each home, including some attributes that normally aren't evaluated and broke evaluation completely. This was relevant for Maya but not for the rest of us - I'm not sure how CI missed it... --- packetmix/modules/nilla-home/nixos.nix | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/packetmix/modules/nilla-home/nixos.nix b/packetmix/modules/nilla-home/nixos.nix index cf43ab46..0a91c533 100644 --- a/packetmix/modules/nilla-home/nixos.nix +++ b/packetmix/modules/nilla-home/nixos.nix @@ -67,7 +67,9 @@ in ( values: let - existingUsernames = builtins.filter (value: value.username != null) values; + existingUsernames = map (value: value.username) ( + builtins.filter (value: value.username != null) values + ); uniqueUsernames = lib.lists.unique existingUsernames; in if existingUsernames != uniqueUsernames then -- 2.43.0