redshift/gammastep: add enableVerboseLogging option

This option controls whether the `-v` command line argument is given. Co-authored-by: Sumner Evans <[email protected]>
wuyoli · Oct 8, 2022 · e1f1160 · e1f1160
1 parent bd87a34
commit e1f1160
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 3 deletions.
diff --git a/modules/services/redshift-gammastep/lib/options.nix b/modules/services/redshift-gammastep/lib/options.nix
@@ -110,6 +110,8 @@ in {
       '';
     };
 
+    enableVerboseLogging = mkEnableOption "verbose service logging";
+
     tray = mkOption {
       type = types.bool;
       default = false;
@@ -199,7 +201,10 @@ in {
         ExecStart = let
           command = if cfg.tray then appletExecutable else mainExecutable;
           configFullPath = config.xdg.configHome + "/${xdgConfigFilePath}";
-        in "${cfg.package}/bin/${command} -v -c ${configFullPath}";
+        in "${cfg.package}/bin/${command} " + cli.toGNUCommandLineShell { } {
+          v = cfg.enableVerboseLogging;
+          c = configFullPath;
+        };
         RestartSec = 3;
         Restart = "on-failure";
       };

diff --git a/tests/modules/services/redshift-gammastep/gammastep-basic-configuration-expected.service b/tests/modules/services/redshift-gammastep/gammastep-basic-configuration-expected.service
@@ -2,7 +2,7 @@
 WantedBy=graphical-session.target
 
 [Service]
-ExecStart=@gammastep@/bin/gammastep -v -c /home/hm-user/.config/gammastep/config.ini
+ExecStart=@gammastep@/bin/gammastep '-c' '/home/hm-user/.config/gammastep/config.ini'
 Restart=on-failure
 RestartSec=3
 

diff --git a/tests/modules/services/redshift-gammastep/redshift-basic-configuration-expected.service b/tests/modules/services/redshift-gammastep/redshift-basic-configuration-expected.service
@@ -2,7 +2,7 @@
 WantedBy=graphical-session.target
 
 [Service]
-ExecStart=@redshift@/bin/redshift -v -c /home/hm-user/.config/redshift/redshift.conf
+ExecStart=@redshift@/bin/redshift '-c' '/home/hm-user/.config/redshift/redshift.conf'
 Restart=on-failure
 RestartSec=3