[bug#41358] lm-sensors: pwmconfig is broken

[bug#41358] lm-sensors: pwmconfig is broken

[Nicolò Balzarotti]

[-- Attachment #1: Type: text/plain, Size: 302 bytes --]

Hello guix!

The current version of lm-sensors adjust grep path in the pwmconfig
script, replacing grep->/gnu/store/.../grep.  However, the script is
using egrep, leading to the non-existant e/gnu/store/.../grep path.
The attached patch fixes is actually replacing egrep->/gnu/store/../egrep

Thanks!


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-gnu-lm-sensors-Use-correct-egrep-path.patch --]
[-- Type: text/x-patch, Size: 1007 bytes --]

From 1a0395630547868385312198659d5fda39845cbe Mon Sep 17 00:00:00 2001
From: nixo <nicolo@nixo.xyz>
Date: Sun, 17 May 2020 14:39:17 +0200
Subject: [PATCH] gnu: lm-sensors: Use correct egrep path.

---
 gnu/packages/linux.scm | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/linux.scm b/gnu/packages/linux.scm
index 290090456f..6dd4fdf747 100644
--- a/gnu/packages/linux.scm
+++ b/gnu/packages/linux.scm
@@ -3541,9 +3541,9 @@ country-specific regulations for the wireless spectrum.")
                (("cat ")
                 (string-append (assoc-ref inputs "coreutils")
                                "/bin/cat "))
-               (("grep ")
+               (("egrep ")
                 (string-append (assoc-ref inputs "grep")
-                               "/bin/grep "))
+                               "/bin/egrep "))
                (("sed -e")
                 (string-append (assoc-ref inputs "sed")
                                "/bin/sed -e"))
-- 
2.26.2

bug#41358: lm-sensors: pwmconfig is broken

[Tobias Geerinckx-Rice via Guix-patches via]

[-- Attachment #1: Type: text/plain, Size: 622 bytes --]

Nicolò,

Nicolò Balzarotti 写道：
> The current version of lm-sensors adjust grep path in the 
> pwmconfig
> script, replacing grep->/gnu/store/.../grep.  However, the 
> script is
> using egrep, leading to the non-existant e/gnu/store/.../grep 
> path.
> The attached patch fixes is actually replacing 
> egrep->/gnu/store/../egrep

Thanks!

As it was, this change would in turn break fancontrol which does 
use plain ‘grep’:

   INTERVAL=$(grep -E '^INTERVAL=.*$' $1 | …

I've modified it to match either and pushed as 
074cd595bbcd343f86aad09794d8781846ff1927.

Kind regards,

T G-R

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 227 bytes --]

[bug#41358] lm-sensors: pwmconfig is broken

[Nicolò Balzarotti]

Hi Tobias!

Tobias Geerinckx-Rice via Guix-patches via <guix-patches@gnu.org>
writes:

> Nicolò,
>
> Nicolò Balzarotti 写道：
>> The current version of lm-sensors adjust grep path in the 
>> pwmconfig
>> script, replacing grep->/gnu/store/.../grep.  However, the 
>> script is
>> using egrep, leading to the non-existant e/gnu/store/.../grep 
>> path.
>> The attached patch fixes is actually replacing 
>> egrep->/gnu/store/../egrep
>
> Thanks!
>
> As it was, this change would in turn break fancontrol which does 
> use plain ‘grep’:
>
>    INTERVAL=$(grep -E '^INTERVAL=.*$' $1 | …
>
Sorry for missing that

> I've modified it to match either and pushed as 
> 074cd595bbcd343f86aad09794d8781846ff1927.
>
Thanks!
Nicolò

> Kind regards,
>
> T G-R