From 127ea2d80e3b08608807909216bc6e46618e4699 Mon Sep 17 00:00:00 2001 From: Philip Taron Date: Thu, 31 Aug 2023 13:40:13 -0700 Subject: [PATCH] htop: fix configure.am error with --enable-sensors Commit 23c81659b5de67f4cf032e31c024a8b1a283fc45 didn't get the configure flag quite right. According to https://github.com/htop-dev/htop/blob/main/configure.ac#L671, it's `--enable-sensors`, not `--with-sensors`. I tested this out locally with ``` programs.htop = { enable = true; package = pkgs.htop.overrideAttrs (prev: { configureFlags = [ "--enable-unicode" "--sysconfdir=/etc" "--enable-affinity" "--enable-capabilities" "--enable-delayacct" "--enable-sensors" ]; }); }; ``` --- pkgs/tools/system/htop/default.nix | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pkgs/tools/system/htop/default.nix b/pkgs/tools/system/htop/default.nix index c00f25780839..50c6ea461f36 100644 --- a/pkgs/tools/system/htop/default.nix +++ b/pkgs/tools/system/htop/default.nix @@ -37,7 +37,7 @@ stdenv.mkDerivation rec { "--enable-capabilities" "--enable-delayacct" ] - ++ lib.optional sensorsSupport "--with-sensors" + ++ lib.optional sensorsSupport "--enable-sensors" ; postFixup =