From 5aafd1d0dba2800e14421b3f93721e8b2d1cd0f7 Mon Sep 17 00:00:00 2001 From: Yarny0 <41838844+Yarny0@users.noreply.github.com> Date: Sat, 13 Jul 2019 09:25:56 +0200 Subject: [PATCH] hplip: udev rule: drop `nohup` instead of patching it hplip's udev rule uses `nohup` to attempt to detach its printer configuration script from the udev process. However, this attempt is futile as udev tracks its child processes with cgroups: https://unix.stackexchange.com/a/243648 The hardcoded path to `nohup` was patched to use Nix' `nohup`. This commit alters the patch script so that it simply drops the call to `nohup`. --- pkgs/misc/drivers/hplip/3.16.11.nix | 2 +- pkgs/misc/drivers/hplip/3.18.5.nix | 2 +- pkgs/misc/drivers/hplip/default.nix | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/pkgs/misc/drivers/hplip/3.16.11.nix b/pkgs/misc/drivers/hplip/3.16.11.nix index 5e275a1f2b3..4e9b4cbf0ae 100644 --- a/pkgs/misc/drivers/hplip/3.16.11.nix +++ b/pkgs/misc/drivers/hplip/3.16.11.nix @@ -176,7 +176,7 @@ pythonPackages.buildPythonApplication { # Needs a lot of patching but might save someone a bit of confusion: substituteInPlace $out/etc/udev/rules.d/56-hpmud.rules \ --replace {,${bash}}/bin/sh \ - --replace {/usr,${coreutils}}/bin/nohup \ + --replace /usr/bin/nohup "" \ --replace {,${utillinux}/bin/}logger \ --replace {/usr,$out}/bin ''; diff --git a/pkgs/misc/drivers/hplip/3.18.5.nix b/pkgs/misc/drivers/hplip/3.18.5.nix index 9c9694edbd4..7da6b2db930 100644 --- a/pkgs/misc/drivers/hplip/3.18.5.nix +++ b/pkgs/misc/drivers/hplip/3.18.5.nix @@ -208,7 +208,7 @@ pythonPackages.buildPythonApplication { # Needs a lot of patching but might save someone a bit of confusion: substituteInPlace $out/etc/udev/rules.d/56-hpmud.rules \ --replace {,${bash}}/bin/sh \ - --replace {/usr,${coreutils}}/bin/nohup \ + --replace /usr/bin/nohup "" \ --replace {,${utillinux}/bin/}logger \ --replace {/usr,$out}/bin ''; diff --git a/pkgs/misc/drivers/hplip/default.nix b/pkgs/misc/drivers/hplip/default.nix index eced20902c6..6c02fa52124 100644 --- a/pkgs/misc/drivers/hplip/default.nix +++ b/pkgs/misc/drivers/hplip/default.nix @@ -215,7 +215,7 @@ pythonPackages.buildPythonApplication { # Needs a lot of patching but might save someone a bit of confusion: substituteInPlace $out/etc/udev/rules.d/56-hpmud.rules \ --replace {,${bash}}/bin/sh \ - --replace {/usr,${coreutils}}/bin/nohup \ + --replace /usr/bin/nohup "" \ --replace {,${utillinux}/bin/}logger \ --replace {/usr,$out}/bin '';