Re: [lm-sensors] sensors-detect path issue under fc5 / 6 -- fix

[Jean Delvare <khali@linux-fr.org>]

On Sun, 4 Mar 2007 11:33:41 +0100, Jean Delvare wrote:
> On Sun, 4 Mar 2007 03:04:42 -0500 (EST), Vince Spinelli wrote:
> > It's a simple one, but it's worth noting.
> > 
> > I'm unawares of other distributions, but under Fedora Core 5 and 6, the
> > i2cdetect command is located in the /usr/sbin directory, not /sbin/ nor
> > /usr/local/sbin.
> 
> That's the case of pretty much all distributions shipping lm-sensors,
> as /usr/local is meant for things added on top of the original
> distribution.
> 
> > As a result, an error message is encountered when running
> > sensors-detect.sh as either root, sudo root, or a normal user.
> 
> There's no such thing as "sensors-detect.sh" in our source package.
> sensors-detect is a perl script.
> 
> If your root user doesn't have /usr/sbin in its $PATH, then your setup
> is seriously broken, I doubt sensors-detect will be the only failing
> tool.
> 
> sensors-detect can no longer be run as non-root, so this case doesn't
> matter.
> 
> The su/sudo case is more problematic though, I can imagine a standard
> user gaining root permissions without getting the $PATH usually
> associated with the root user.
> 
> > The fix is as follows... inside of the sensors-detect.sh script, there's a
> > segment to add directories to path.

I've made a functionally equivalent change to sensors-detect, committed
to our SVN repository. Feel free to give it a try. Hopefully it will
make the packagers happier.

-- 
Jean Delvare

_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors