GNU bug report logs - #41358
lm-sensors: pwmconfig is broken

Previous Next

Package: guix-patches;

Reported by: anothersms <at> gmail.com (Nicolò Balzarotti)

Date: Sun, 17 May 2020 12:47:01 UTC

Severity: normal

Done: Tobias Geerinckx-Rice <me <at> tobias.gr>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: Nicolò Balzarotti <anothersms <at> gmail.com>
To: me <at> tobias.gr, 41358 <at> debbugs.gnu.org
Cc: 41358-done <at> debbugs.gnu.org
Subject: [bug#41358] lm-sensors: pwmconfig is broken
Date: Sun, 17 May 2020 17:19:44 +0200
Hi Tobias!

Tobias Geerinckx-Rice via Guix-patches via <guix-patches <at> 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




This bug report was last modified 5 years and 2 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.