From: Duncan W. <du...@fr...> - 2008-04-25 20:22:59
|
Torsten Kurbad wrote: > On Friday, April 25, 2008 at 07:13 Duncan Webb wrote: > > >>> Provided SENSORS_PLATFORM_PATH exists, the getSensorPath(...) method in >>> plugins.idlebar.system.sensors simply returns _some_ existing directory >>> from that path. >>> > > >> What about using the path: /sys/class/hwmon/hwmon0/device/ >> > > that looks much too "dynamic" in my eyes. What, if, e.g., the coretemp > module accidentely becomes "hwmon0"? Also, the plugin as it is, assumes > the tempX_Y files in a _subdirectory_ of PLATFORM_PATH, therefore we'd > have to provide "/sys/class/hwmon/hwmon0", and the "/device" subdir gets > merely chosen by accident. > IIRC, it's described in lm_sensors docs that /sys/class/hwmon/hwmon0 is the correct place. > I filed a bugreport on this, maybe some people (even those with > currently working setup) are willing to test the patch to see if it > breaks anything... > Duncan |