Hi Mike, On Thu, Jan 12, 2012 at 10:52:57AM -0500, Mike Edwards wrote: > Looked at the patch you provided to upstream. There's a small typo - > the test checks for generation>=2 || (generation==2 && major>=5). The > first generation check should be 3: > if (strcmp(p_uname->sysname, "Linux")==0 && (generation>=3 || > (generation==2 && major>=5))) >
Meh, indeed, stupid typo from me (anyway, it's harmless, unless you have a 2.4 kernel ;-). I updated the patch attached upstream, and also the patch in our packaging, it should be part of the next upload. Thanks! > With that small change, I was able to see the hdd temperature sensor > again. > > Thanks again, Lionel! > > > > On Sat, Dec 24, 2011 at 01:52:01PM +0100, Lionel Le Folgoc babbled thus: > > tags 640052 + confirmed upstream > > forwarded 640052 https://bugzilla.xfce.org/show_bug.cgi?id=8266 > > thanks > > > > Hi, > > > > Thanks for the follow up. > > > > On Tue, Dec 20, 2011 at 07:54:17PM -0500, Mike Edwards wrote: > > > Package: xfce4-sensors-plugin > > > Version: 1.2.3-1+b1 > > > Followup-For: Bug #640052 > > > > > > Dear Maintainer, > > > > > > * What led up to the situation? > > > Upgrade to Wheezy > > > > > > * What exactly did you do (or not do) that was effective (or > > > ineffective)? > > > Ensured hddtemp was mode u+s, is currently running, and returns valid > > > data. > > > > > > * What was the outcome of this action? > > > Nothing. xfce4-sensors-plugin simply does not offer the option of > > > displaying > > > hdd temps. 1.0.0-1+b1 from squeeze supports this functionality. > > > > > > * What outcome did you expect instead? > > > The option to display the temperatures of any connected hard disks. > > > > > > I'd like to confirm that this bug is present. This version of > > > xfce4-sensors-plugins will not use hddtemp no matter what I do. > > > > > > > If you still have a 2.6.x kernel around, can you try to reboot on it and > > confirm that hdd temps are correctly displayed? > > > > I found a buggy test in the panel plugin (it checks for a X.Y kernel > > version, with Y >= 5, which obviously doesn't work on a 3.0/3.1 kernel > > in wheezy…). Fixing this test let the temps reappear for me. The patch > > will be part of the next upload. > > > > -- > > Regards, and merry Christmas, > > Lionel > > > > -- > > Mike Edwards | If this email address disappears, > Unsolicited advertisments to | assume it was spammed to death. To > this address are not welcome. | reach me in that case, s/-.*@/@/ > > "Our progress as a nation can be no swifter than our progress in education. > The human mind is our fundamental resource." > -- John F. Kennedy -- Regards, Lionel
signature.asc
Description: Digital signature
