Skip to content

Commit 8986591

Browse files
authored
Merge branch 'nix-darwin:master' into master
2 parents 0587990 + 830b3f0 commit 8986591

File tree

5 files changed

+15
-8
lines changed

5 files changed

+15
-8
lines changed

modules/launchd/launchd.nix

Lines changed: 11 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -121,7 +121,17 @@ in
121121
key. Please note: many people are confused by this key. Please read `execvp(3)` very carefully!
122122
'';
123123
# TODO: Remove this some time after 25.11.
124-
apply = map (value: lib.warnIf (lib.hasInfix "&" value) "A value for `ProgramArguments` contains the literal string `&`. This is no longer necessary and will lead to double-escaping, as nix-darwin now automatically escapes special characters." value);
124+
apply =
125+
val:
126+
if (builtins.isNull val) then
127+
val
128+
else
129+
(map (
130+
item:
131+
lib.warnIf (lib.hasInfix "&" item)
132+
"A value for `ProgramArguments` contains the literal string `&`. This is no longer necessary and will lead to double-escaping, as nix-darwin now automatically escapes special characters."
133+
item
134+
) val);
125135
};
126136

127137
EnableGlobbing = mkOption {

modules/nix/default.nix

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -835,7 +835,7 @@ in
835835

836836
system.requiresPrimaryUser = mkIf (
837837
config.system.stateVersion < 2
838-
&& options.nix.nixPath.highestPrio == (mkDefault {}).priotity
838+
&& options.nix.nixPath.highestPrio == (mkDefault {}).priority
839839
) [
840840
"nix.nixPath"
841841
];

modules/services/github-runner/options.nix

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -262,8 +262,8 @@ in
262262
};
263263

264264
nodeRuntimes = mkOption {
265-
type = with types; nonEmptyListOf (enum [ "node20" ]);
266-
default = [ "node20" ];
265+
type = with types; nonEmptyListOf (enum [ "node20" "node24" ]);
266+
default = [ "node20" "node24" ];
267267
description = ''
268268
List of Node.js runtimes the runner should support.
269269
'';

modules/services/github-runner/service.nix

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,7 @@ in
7676

7777
config.launchd.daemons = flip mapAttrs' config.services.github-runners (name: cfg:
7878
let
79-
package = cfg.package.override (old: optionalAttrs (hasAttr "nodeRuntimes" old) { inherit (cfg) nodeRuntimes; });
79+
package = cfg.package.override { inherit (cfg) nodeRuntimes; };
8080
stateDir = mkStateDir cfg;
8181
logDir = mkLogDir cfg;
8282
workDir = mkWorkDir cfg;

tests/services-github-runners.nix

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,9 +4,6 @@
44
enable = true;
55
url = "https://github.com/nixos/nixpkgs";
66
tokenFile = "/secret/path/to/a/github/token";
7-
# We need an overridable derivation but cannot use the actual github-runner package
8-
# since it still relies on Node.js 16 which is marked as insecure.
9-
package = pkgs.hello;
107
};
118

129
test = ''

0 commit comments

Comments
 (0)