From e1fab012e872c129099535b5b535fc2347cfa6f4 Mon Sep 17 00:00:00 2001
From: toonn <toonn@toonn.io>
Date: Fri, 1 Apr 2022 17:33:07 +0200
Subject: [PATCH] nix-darwin: sudo --set-home for multiple user activation
 (#2857)

Changing from `sudo -i` to `sudo -s` messes up activation when multiple
users are managed. `--set-home` should have similar behavior to `-i` in
that the activation script is run from the user's home directory.

Fixes #2856
---
 nix-darwin/default.nix | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/nix-darwin/default.nix b/nix-darwin/default.nix
index a56b3cdd6..007e2de38 100644
--- a/nix-darwin/default.nix
+++ b/nix-darwin/default.nix
@@ -134,7 +134,7 @@ in
     system.activationScripts.postActivation.text =
       concatStringsSep "\n" (mapAttrsToList (username: usercfg: ''
         echo Activating home-manager configuration for ${username}
-        sudo -u ${username} -s ${pkgs.writeShellScript "activation-${username}" ''
+        sudo -u ${username} -s --set-home ${pkgs.writeShellScript "activation-${username}" ''
           ${lib.optionalString (cfg.backupFileExtension != null)
             "export HOME_MANAGER_BACKUP_EXT=${lib.escapeShellArg cfg.backupFileExtension}"}
           ${lib.optionalString cfg.verbose "export VERBOSE=1"}