Hi, a Debian user requested that sensor plugin would not warn every time about missing setuid stuff for hddtemp. If it's not configured with setuid bit, sensors should warn the fist time, but ignore it completely the following times. Cheers, -- Yves-Alexis
And if hddtemp is not present at all, same thing, the plugin could warn once, but only once.
there you go, this also fixed some null pointer problems.
Wanna close this one, too?
"there you go, this also fixed some null pointer problems."
Seems this one is back (since a few months/years, actually, so reopening).
(In reply to comment #5) > Seems this one is back (since a few months/years, actually, so reopening). You may know that you can build without hddtemp support. What version do you use? There hasn't been any new source commit since a long time, last: 2012-12-28 20:51:53 (GMT).
(In reply to comment #6) > (In reply to comment #5) > > Seems this one is back (since a few months/years, actually, so reopening). > > You may know that you can build without hddtemp support. I know. We build for all the distribution users, not only one people, so we do enable it at build time. What we need is a way to have it disabled at runtime. Exactly what this bug report was about. The fix was committed, but there was a regression somehow at one point. > What version do > you use? There hasn't been any new source commit since a long time, last: > 2012-12-28 20:51:53 (GMT). 1.2.5.
(In reply to Yves-Alexis Perez from comment #7) > (In reply to comment #6) > > (In reply to comment #5) > > > Seems this one is back (since a few months/years, actually, so reopening). > > > > You may know that you can build without hddtemp support. > > I know. We build for all the distribution users, not only one people, so we > do enable it at build time. What we need is a way to have it disabled at > runtime. > > Exactly what this bug report was about. The fix was committed, but there was > a regression somehow at one point. > > > What version do > > you use? There hasn't been any new source commit since a long time, last: > > 2012-12-28 20:51:53 (GMT). > > 1.2.5. OK, what "warnings" are found? The current code intendedly alwys showed the notifications, but it was possible to shutdown the warnings dialog. Gonna change it to only display the notification once, but for any user missing it, there will be trouble helping to track down issues with hddtemp.
commit bcad538..4ada645 : suppress option is now also respected for notifications as well as for dialogs.