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
