From a9033bdaa9ab490935ba97fe28b3dcdf195b139d 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 | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packetmix/modules/nilla-home/nixos.nix b/packetmix/modules/nilla-home/nixos.nix index cf43ab46..d1cb2fde 100644 --- a/packetmix/modules/nilla-home/nixos.nix +++ b/packetmix/modules/nilla-home/nixos.nix @@ -67,7 +67,7 @@ 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