Re: [Freevo-devel] Patch for idlebar/system in Freevo-1.8.0

2008-04-27 Thread Torsten Kurbad
Hi, >> If I find the time, I'll look into the module and see, how it could be >> done more dynamically. E.g., I'd rather like to see three temperatures: >> >> system, coretemp0, coretemp1 >> >> so maybe the module could be restructured to accept an arbitrary number >> of tuples: ('sensorname', '

Re: [Freevo-devel] Patch for idlebar/system in Freevo-1.8.0

2008-04-27 Thread Duncan Webb
Torsten Kurbad wrote: >>> On Friday, April 25, 2008 at 22:22 Duncan Webb wrote: >> I was just trying to say where the correct path now is. As of 2.6.24 >> there does not seem to be any sensors in the /sys/devices/platform/ path. > > I see. My uname -r says: > > # 2.6.24-gentoo-r3 > > and /sys/de

Re: [Freevo-devel] Patch for idlebar/system in Freevo-1.8.0

2008-04-26 Thread Torsten Kurbad
>> On Friday, April 25, 2008 at 22:22 Duncan Webb wrote: > Sorry I didn't mean that you patch was in any way invalid. No harm done! :o) > I was just trying to say where the correct path now is. As of 2.6.24 > there does not seem to be any sensors in the /sys/devices/platform/ path. I see. My una

Re: [Freevo-devel] Patch for idlebar/system in Freevo-1.8.0

2008-04-26 Thread Duncan Webb
Torsten Kurbad wrote: > On Friday, April 25, 2008 at 22:22 Duncan Webb wrote: > 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

Re: [Freevo-devel] Patch for idlebar/system in Freevo-1.8.0

2008-04-25 Thread Torsten Kurbad
On Friday, April 25, 2008 at 22:22 Duncan Webb wrote: >>> 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 _subdir

Re: [Freevo-devel] Patch for idlebar/system in Freevo-1.8.0

2008-04-25 Thread Duncan Webb
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

Re: [Freevo-devel] Patch for idlebar/system in Freevo-1.8.0

2008-04-24 Thread Duncan Webb
On 24/04/2008 18:43, Torsten Kurbad said the following: > Folks, > > today I upgraded from 1.7.6.1 to 1.8.0. Everything went very smoothly - > nice work! > > But I discovered a little bug, that the 1.7 series has as well: > > Provided SENSORS_PLATFORM_PATH exists, the getSensorPath(...) method i