From 1315f934711210d2dc1b0a5edf111cb817ad7ad6 Mon Sep 17 00:00:00 2001
From: Colin <colin@uninsane.org>
Date: Wed, 2 Oct 2024 14:30:32 +0000
Subject: [PATCH] users: dont restart systemd services on activation

i can see why this would be desired... but unless i plumb a way to selectively disable it, i'd prefer to disable it by default
---
 TODO.md                   | 2 --
 modules/users/systemd.nix | 3 ++-
 2 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/TODO.md b/TODO.md
index db8af9e3b..ac5646858 100644
--- a/TODO.md
+++ b/TODO.md
@@ -1,6 +1,4 @@
 ## BUGS
-- sway & other services are restart when NixOS config changes
-  - probably want to just not restart any user services by default
 - nmcli and networkmanager_dmenu don't have the right perms to adjust networking details
   - i think the error is in my sandboxing of the NetworkManager service
 - `rmDbusServices` may break sandboxing
diff --git a/modules/users/systemd.nix b/modules/users/systemd.nix
index 53eb08ffd..554855251 100644
--- a/modules/users/systemd.nix
+++ b/modules/users/systemd.nix
@@ -28,7 +28,7 @@ let
     # documentation,
     # depends,
     # dependencyOf,
-    partOf,
+    # partOf,
     command,
     cleanupCommand,
     startCommand,
@@ -43,6 +43,7 @@ let
       serviceConfig.User = userName;
       serviceConfig.Group = "users";
       serviceConfig.WorkingDirectory = "~";
+      serviceConfig.X-RestartIfChanged = lib.mkDefault false;  #< NixOS attribute, so we don't restart the DE on activation
 
       serviceConfig.ExecSearchPath = [
         "/etc/profiles/per-user/${userName}/bin"