Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 449470 - sys-apps/lm_sensors-3.3.3-r1 init.d/sensord tries to run /usr/sbin/sensorsd instead of /usr/sbin/sensord
Summary: sys-apps/lm_sensors-3.3.3-r1 init.d/sensord tries to run /usr/sbin/sensorsd i...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Core system (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Jeff (JD) Horelick (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-12-31 15:52 UTC by Steven Green
Modified: 2013-09-17 20:04 UTC (History)
3 users (show)

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

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Steven Green 2012-12-31 15:52:56 UTC 
With the sensord use flag, lm_sensors installs /etc/init.d/sensord from sensord-4-init.d

sensord-4-init.d includes the line:
command=/usr/sbin/sensorsd

This should be:
command=/usr/sbin/sensord




Reproducible: Always

Steps to Reproduce:
1. emerge lm_sensors-3.3.3-r1 with sensord use flag.
2. /etc/init.d/sensord start
3. Note that it fails to start because /usr/sbin/sensorsd does not exist.
Actual Results:  
sudo /etc/init.d/sensord start
 * Starting sensord ...
 * start-stop-daemon: /usr/sbin/sensorsd does not exist
 * Failed to start sensord                                                                    [ !! ]
 * ERROR: sensord failed to start
Comment 1 Todd Walter 2013-01-31 01:27:54 UTC 
Behaviour is reproducible on my system.  Proposed change works.
Comment 2 Chris Mayo 2013-02-02 12:22:54 UTC 
Fixed in:

*lm_sensors-3.3.3-r2 (31 Jan 2013)

  31 Jan 2013; Diego E. Pettenò <flameeyes@gentoo.org>
  +lm_sensors-3.3.3-r2.ebuild, -lm_sensors-3.3.3-r1.ebuild,
  -lm_sensors-3.3.3.ebuild, files/sensord-4-init.d:
  Fix sensord init script that was typoed (thanks to Gordon Pritchard for the
  report); make it use /run directly, overriding the default pid file location.
  Remove unused for the slot.

but N.B. Bug 455120