Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bugzilla DB migration completed. Please report issues to Infra team via email via infra@gentoo.org or IRC
Bug 28137 - lm-sensors files/rc - /etc/init.d/lm_sensors - doesn't allow modules to be loaded with arguments or options in /etc/conf.d/lm_sensors
Summary: lm-sensors files/rc - /etc/init.d/lm_sensors - doesn't allow modules to be lo...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High minor (vote)
Assignee: Tim Yamin (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-09-07 17:58 UTC by chkno
Modified: 2004-02-10 13:13 UTC (History)
0 users

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


Attachments
Patch to lm-sensors/files/rc (lm-sensors-files-rc.patch,387 bytes, patch)
2003-09-07 18:00 UTC, chkno
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description chkno 2003-09-07 17:58:53 UTC
If you add options to modules in /etc/conf.d/lm_sensors, eg: 
MODULE_0="w83781d init=0 force_subclients=0,0x2c,0x4a,0x4b" 
 
/etc/init.d/lm_sensors loads it properly, but makes red exclamation marks on unload. 

Reproducible: Always
Steps to Reproduce:
1. Add arguments to a MODULE_# line in /etc/conf.d/lm_sensors 
2. Run "/etc/init.d/lm_sensors start".  It passes. 
3. Run "/etc/init.d/lm_sensors stop".  It fails. 
Actual Results:  
 *   Removing w83781d init=0 force_subclients=0,0x2c,0x4a,0x4b...         [ !! ] 
 

Expected Results:  
 *   Removing w83781d...                                                  [ ok ]
Comment 1 chkno 2003-09-07 18:00:22 UTC
Created attachment 17242 [details, diff]
Patch to lm-sensors/files/rc
Comment 2 Brandon Low (RETIRED) gentoo-dev 2004-02-10 12:53:45 UTC
reassigning to plasmaroo as he maintains sensors now.
Comment 3 Tim Yamin (RETIRED) gentoo-dev 2004-02-10 13:13:27 UTC
Use MODULE_X_ARGS in /etc/conf.d/lm_sensors and it'll work fine. I've updated the patches in CVS, adding that message along to the explanation comments. Thanks.