bug#41358: closed (lm-sensors: pwmconfig is broken)

[GNU bug Tracking System]

Your message dated Sun, 17 May 2020 17:12:07 +0200
with message-id <877dxavbnc.fsf@nckx>
and subject line Re: [bug#41358] lm-sensors: pwmconfig is broken
has caused the debbugs.gnu.org bug report #41358,
regarding lm-sensors: pwmconfig is broken
to be marked as done.

(If you believe you have received this mail in error, please contact
address@hidden.)


-- 
41358: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=41358
GNU Bug Tracking System
Contact address@hidden with problems

--- Begin Message --- Subject: lm-sensors: pwmconfig is broken Date: Sun, 17 May 2020 14:46:25 +0200

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!

0001-gnu-lm-sensors-Use-correct-egrep-path.patch
Description: Text Data

--- End Message ---

--- Begin Message --- Subject: Re: [bug#41358] lm-sensors: pwmconfig is broken Date: Sun, 17 May 2020 17:12:07 +0200

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

signature.asc
Description: PGP signature

--- End Message ---