Skip to content

Commit

Permalink
Update nixos-module.nix (#747)
Browse files Browse the repository at this point in the history 
flipping serviceConfig.Restart to "yes"
  • Loading branch information
@andreyadrian
andreyadrian committed Oct 16, 2023
1 parent 8440eaf commit 1b79c5d
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion nix/nixos-module.nix
View file
Original file line number Diff line number Diff line change
Expand Up @@ -126,7 +126,7 @@ let
value = {
description = "KMonad for ${keyboard.device}";
script = lib.escapeShellArgs cmd;
serviceConfig.Restart = "no";
serviceConfig.Restart = "yes";
serviceConfig.User = "kmonad";
serviceConfig.SupplementaryGroups = groups;
serviceConfig.Nice = -20;
Expand Down

2 comments on commit 1b79c5d

@akho
Copy link

@akho akho commented on 1b79c5d Oct 17, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Restart doesn’t have a "yes" value, and the module doesn’t work now.

in man systemd.service;

Takes one of no, on-success, on-failure, on-abnormal, on-watchdog, on-abort, or always.

This needs to be changed to always.

@slotThe
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sigh sorry about that, I should have vetted this more thoroughly. Fixed by a1aea16

Thanks for reporting!

Please sign in to comment.