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

-------------------------------------------------------------------------
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
_______________________________________________
Freevo-devel mailing list
Freevo-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/freevo-devel

Reply via email to