From 46f93825af684a094950ae66ba5ef24a4b10c49f Mon Sep 17 00:00:00 2001
From: Evyatar Stalinsky <evyatar@stalinsky.xyz>
Date: Sat, 5 Apr 2025 13:33:12 +0300
Subject: [PATCH] redshift/gammastep: fix tray.target dependency

Fix a missing dependency for `tray.target` in `redshift/gammastep` module when the tray feature is enabled.

This caused a race condition that could result in the redshift/gammastep systemd user unit failing because the tray provider (e.g. waybar) isn't ready yet.
---
 modules/services/redshift-gammastep/lib/options.nix | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/modules/services/redshift-gammastep/lib/options.nix b/modules/services/redshift-gammastep/lib/options.nix
index 476f8baa3..d53ea03c6 100644
--- a/modules/services/redshift-gammastep/lib/options.nix
+++ b/modules/services/redshift-gammastep/lib/options.nix
@@ -189,8 +189,10 @@ in {
       in {
         Description = "${programName} colour temperature adjuster";
         Documentation = serviceDocumentation;
-        After = [ "graphical-session.target" ] ++ geoclueAgentService;
+        After = [ "graphical-session.target" ]
+          ++ (lib.optional cfg.tray "tray.target") ++ geoclueAgentService;
         Wants = geoclueAgentService;
+        Requires = lib.mkIf cfg.tray "tray.target";
         PartOf = [ "graphical-session.target" ];
       };