From 0d3fdbdb54f521952e35b1c6a3612484f6643610 Mon Sep 17 00:00:00 2001
From: Qyriad <qyriad@qyriad.me>
Date: Fri, 9 Jan 2026 17:46:05 +0100
Subject: [PATCH] Exorcise MORE Darwin test demons because functional1 is
 /awful/

Yeah uh, I don't think `$shell` is ever set in the functional1
environment. Pretty sure this has been expanding to `''` THIS WHOLE
TIME.

Also `SHELL` is `/sbin/nologin` in that builder. idk why so I'm just
passing `shell` unconditionally in functional's `mkDerivation` helper.

Maybe next we should put `set -euo pipefail` into all these test
builders?

Change-Id: Ie5fb85305932c03a901ba84ffcce48f16a6a6964
---
 tests/functional/config.nix.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/functional/config.nix.in b/tests/functional/config.nix.in
index 70b4d53fe..dd4b70f49 100644
--- a/tests/functional/config.nix.in
+++ b/tests/functional/config.nix.in
@@ -24,7 +24,7 @@ rec {
 
   mkDerivation = args:
     derivation ({
-      inherit system;
+      inherit system shell;
       builder = shell;
       args = ["-e" args.builder or (builtins.toFile "builder-${args.name}.sh" ''
         if [ -e "$NIX_ATTRS_SH_FILE" ]; then source $NIX_ATTRS_SH_FILE; fi;