Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 399321 - sys-apps/lm_sensors-3.3.1 - sensors-detect fails to write configuration file
Summary: sys-apps/lm_sensors-3.3.1 - sensors-detect fails to write configuration file
Status: RESOLVED INVALID
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Core system (show other bugs)
Hardware: All All
: Normal normal (vote)
Assignee: Lars Wendler (Polynomial-C) (RETIRED)
URL:
Whiteboard:
Keywords: NeedPatch
Depends on:
Blocks:
 
Reported: 2012-01-18 21:22 UTC by Jaak Ristioja
Modified: 2016-09-16 12:52 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Jaak Ristioja 2012-01-18 21:22:35 UTC
After all detection procedures, sensors-detect exits successfully with the message "No modules to load, skipping modules configuration." and generates no /etc/conf.d/lm_sensors file.

I guess the following code in /usr/sbin/sensors-detect is to blame:

    if (@{$hwmon_modules}) {
        write_config($configfile, $bus_modules, $hwmon_modules);
    } else {
        print "No modules to load, skipping modules configuration.\n\n";
    }

Please fix. Thanks!
Comment 1 Antonio Regidor Garcia 2012-06-30 11:24:54 UTC
I think I have the same problem. If I compile coretemp as a kernel module, sensors-detect detects it and creates /etc/conf.d/lm_sensor file, but if I build coretemp into the kernel, sensors-detect doesn't create the file.
Comment 2 Sergey Popov gentoo-dev 2012-11-19 16:52:09 UTC
This is normal behaviour for lm_sensors, cause initscript for it just load modules and if you have modules compiled in kernel - there is nothing to load. If you think that this behaviour is wrong - file bug to upstream, not here.
Comment 3 Sergey Popov gentoo-dev 2012-11-19 16:57:02 UTC
Oops, i was too hurry with conclusions, my bad. lm_sensors package uses Gentoo-specific patch for configuration building, so it may be our problem.
Comment 4 Thomas Deutschmann (RETIRED) gentoo-dev 2016-09-16 12:52:17 UTC
If all your modules are build into your kernel there's nothing sensors-detect can do and no need to load modules.

So pinkbyte was right with his first classification.

If you still think we are wrong and there's something to fix please file a new bug against latest sys-apps/lm_sensors-3.4.0_p20160725 package where we have rewritten Gentoo's custom sensors-detect (well, it doesn't address the reported problem because from my view there's nothing to address)...

Thanks!