From f6e1bf7fd3bd83a2c0656886c77433dcbaef12e6 Mon Sep 17 00:00:00 2001 From: Yarny0 <41838844+Yarny0@users.noreply.github.com> Date: Tue, 22 Sep 2026 19:44:28 +0200 Subject: [PATCH] nixos/printers: fix shellcheck issue again Some arguments to `lpadmin` might look like shell expressions, triggering shellcheck warnings. This was fixed earlier already, in https://github.com/NixOS/nixpkgs/pull/370909/commits/6bb5aa2d26b12f984d9d35840a064268d4530c8e , but the fix got lost later, in https://github.com/NixOS/nixpkgs/pull/558981/commits/85e3dd1d485aa149371a81bd7055380cd19edf3b . --- nixos/modules/hardware/printers.nix | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/nixos/modules/hardware/printers.nix b/nixos/modules/hardware/printers.nix index ce63a9a98980..7af7db9e3630 100644 --- a/nixos/modules/hardware/printers.nix +++ b/nixos/modules/hardware/printers.nix @@ -318,7 +318,10 @@ in "x" ]; in - "lpadmin ${toLpadminArgs argsDpx} ${toLpadminArgs argsWoDpx}"; + '' + # shellcheck disable=SC2016 + lpadmin ${toLpadminArgs argsDpx} ${toLpadminArgs argsWoDpx} + ''; in ''