[Pkg-xfce-devel] Bug#640052: Bug#640052: xfce4-sensors-plugin: No hdd temperature

[Lionel Le Folgoc]

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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.alioth.debian.org/pipermail/pkg-xfce-devel/attachments/20120121/05f170cf/attachment.pgp>