Bug 76357 – Add NOINITSENSORS flag to lm_sensors

• Product: Gentoo Linux
• Status: RESOLVED FIXED

[Description Colin Tinker 2005-01-01 17:12:55 UTC]

LOADMODULES=yes is required for the new /etc/init.d/lm_sensors to load the required modules.  Either the ebuild should warn about this or the sensors-detect should add it

Reproducible: Always
Steps to Reproduce:
1.run sensors-detect
2.
3.

Actual Results:  
LOADMODULES=yes not added to /etc/conf.d/lm_sensors 

Expected Results:  
LOADMODULES=yes added to /etc/conf.d/lm_sensors

[Comment 1 Stefan Briesenick (RETIRED) 2005-01-02 17:34:33 UTC]

dito here.

But I would furthermore suggest to add a

INITSENSORS=yes

for enable/disable the following part:

> if [ "$INITSENSORS" == "yes" ] ; then

    if ! [ -f /etc/sensors.conf ] ; then
            eerror "/etc/sensors.conf does not exist!"
            return 1
    fi

    ebegin "Initializing sensors"
    /usr/bin/sensors -s &>/dev/null
    eend $?

> fi


I have some machines, where "sensors -s" produces strange results (raise fan @ 6000 rpm or contant beeping), whereas the modules are working perfectly. I want to have a possibility to disable the "sensors -s" part.

[Comment 2 Tim Yamin (RETIRED) 2005-01-16 06:27:01 UTC]

LOADMODULES has been changed to NOLOADMODULES, closing bug as FIXED. If you still get this issue after an emerge sync and remerge of lm-sensors please reopen this bug. Thanks!

[Comment 3 Stefan Briesenick (RETIRED) 2005-01-16 13:48:57 UTC]

hey, what about a "NOINITSENSORS"?

I need this flag! Look at Comment #1.

Should be easy to implement. Thanks!

[Comment 4 Stefan Briesenick (RETIRED) 2005-01-16 13:55:02 UTC]

I forgot the ;-)

[Comment 5 Tim Yamin (RETIRED) 2005-01-16 14:37:00 UTC]

Reopen :-]

[Comment 6 Stefan Briesenick (RETIRED) 2005-01-16 15:13:27 UTC]

ty. ;-)

[Comment 7 Henrik Brix Andersen 2005-06-04 07:30:12 UTC]

This will be added in the next release.

[Comment 8 Henrik Brix Andersen 2005-06-11 04:30:26 UTC]

Fixed in sys-apps/lm_sensors-2.9.1. Thank you for reporting this.