Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 98753 - Add fancontol init script to lm_sensors
Summary: Add fancontol init script to lm_sensors
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Unspecified (show other bugs)
Hardware: All Linux
: High enhancement (vote)
Assignee: Mobile Herd (OBSOLETE)
URL:
Whiteboard:
Keywords: InVCS
Depends on:
Blocks:
 
Reported: 2005-07-12 02:33 UTC by Michal Pytasz
Modified: 2006-02-15 12:30 UTC (History)
0 users

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 Michal Pytasz 2005-07-12 02:33:48 UTC
Prepared an /etc/init.d/fancontrold - if there isn't one so far

Reproducible: Always
Steps to Reproduce:
1.
2.
3.

Actual Results:  
Works for me...


#!/sbin/runscript

FANPID=/var/run/fancontrol.pid
FANCONTROL=/usr/sbin/fancontrol

depend() {
        after lm_sensors
}

start() {
        ebegin "Starting fan control daaemon"
        start-stop-daemon --start --quiet --pidfile $FANPID --exec $FANCONTROL 
-- >>/dev/null &
        eend $?
}

stop() {
        ebegin "Stopping fan control daaemon"
        start-stop-daemon --stop --quiet --pidfile $FANPID
        eend $?
}
Comment 1 Sergey Belyashov 2005-08-26 12:35:31 UTC
I think that in need to insert "--background" and remove "&" from line:
start-stop-daemon --start --quiet --pidfile $FANPID --exec $FANCONTROL
-- >>/dev/null &

Result:
start-stop-daemon --start --quiet  --background \
--pidfile $FANPID --exec $FANCONTROL -- >>/dev/null
Comment 2 Henrik Brix Andersen 2005-10-27 12:44:52 UTC
I plan to include this with the next revision/version bump. Thanks.
Comment 3 Henrik Brix Andersen 2006-02-15 10:42:11 UTC
Included a slightly modified version of the init script in sys-apps/lm_sensors-2.10.0. Thank you for contributing this script :)

Since I have no way of verifying that the script works (lack of supported hardware), please verify this bug report when you have tested sys-apps/lm_sensors-2.10.0.
Comment 4 Michal Pytasz 2006-02-15 11:58:10 UTC
Well,

What I got was:

# /etc/init.d/fancontrol start
 * Caching service dependencies ...                                                                                                 [ ok ]
 * Starting fancontrol ...
/sbin/start-stop-daemon: unrecognized option `--pid-file'
Try `/sbin/start-stop-daemon --help' for more information.                                                                          [ !! ]
Comment 5 Michal Pytasz 2006-02-15 12:00:54 UTC
Pressed commit too early, sorry.

You seem to have spelled - it should be --pidfile (one word) not --pid-file
This way it does work.
Comment 6 Henrik Brix Andersen 2006-02-15 12:30:32 UTC
Oops, sorry. Fixed in CVS. Thanks.