tag 507760 patch
thanks

Cyril Brulebois <[EMAIL PROTECTED]> (04/12/2008):
> I might have a look at the history between both versions to see what
> caused this.

Bleh, huge diff. :)

Anyway, first guess was correct, the following (source) debdiff allows
me to get the same results as with 1.8.8-3.

I'll leave it to you to ask upstream why/whether that code deletion was
on purpose/needed for some reasons. :)

Cheers,
-- 
Cyril Brulebois
diff -u ipmitool-1.8.9/debian/changelog ipmitool-1.8.9/debian/changelog
--- ipmitool-1.8.9/debian/changelog
+++ ipmitool-1.8.9/debian/changelog
@@ -1,3 +1,11 @@
+ipmitool (1.8.9-2) unstable; urgency=low
+
+  * Make sdr/Temperatures reading working again (Closes: #507760):
+     - Add patch to revert some bits from 1.8.8 to 1.8.9:
+       20_ipmi_sdr.dpatch
+
+ -- Cyril Brulebois <[EMAIL PROTECTED]>  Thu, 04 Dec 2008 11:30:23 +0100
+
 ipmitool (1.8.9-1) unstable; urgency=low
 
   * Adopt package (Closes: #432027)
diff -u ipmitool-1.8.9/debian/patches/00list 
ipmitool-1.8.9/debian/patches/00list
--- ipmitool-1.8.9/debian/patches/00list
+++ ipmitool-1.8.9/debian/patches/00list
@@ -3,0 +4 @@
+20_ipmi_sdr.dpatch
only in patch2:
unchanged:
--- ipmitool-1.8.9.orig/debian/patches/20_ipmi_sdr.dpatch
+++ ipmitool-1.8.9/debian/patches/20_ipmi_sdr.dpatch
@@ -0,0 +1,24 @@
+#! /bin/sh /usr/share/dpatch/dpatch-run
+## 20_ipmi_sdr.dpatch by Cyril Brulebois <[EMAIL PROTECTED]>
+##
+## All lines beginning with `## DP:' are a description of the patch.
+## DP: Enable reading temperatures again (Closes: #507760).
+
[EMAIL PROTECTED]@
+
+--- a/lib/ipmi_sdr.c
++++ b/lib/ipmi_sdr.c
+@@ -1122,6 +1122,13 @@
+               } else if (IS_SCANNING_DISABLED(rsp->data[1])) {
+                       /* Sensor Scanning Disabled */
+                       validread = 0;
++                      if (rsp->data[0] != 0) {
++                              /* we might still get a valid reading */
++                              val = sdr_convert_sensor_reading(sensor,
++                                                               rsp->data[0]);
++                              if (val != 0.0)
++                                      validread = 1;
++                      }
+               } else if (rsp->data[0] != 0) {
+                       /* convert RAW reading into units */
+                       val = sdr_convert_sensor_reading(sensor, rsp->data[0]);

Attachment: signature.asc
Description: Digital signature

Reply via email to