All the patches except one are correct and applied.  Thanks for the
fixes.  I need to get a formal release out soon, and this was a good
addition.

The following patch is incorrect.  It was caught by some automatic
printf checker, but the call here doesn't exactly match printf
semantics.  So please remove this one.

Thanks again,

-corey

diff -wruN -x '*~' -x '*.o' -x '*.a' -x '*.so' -x '*.so.[0-9]' -x 
autom4te.cache -x .deps -x .libs ../orig-OpenIPMI-2.0.20-rc3/swig/OpenIPMI.i 
./swig/OpenIPMI.i
--- ../orig-OpenIPMI-2.0.20-rc3/swig/OpenIPMI.i 2013-08-28 17:02:23.000000000 
+0200
+++ ./swig/OpenIPMI.i   2013-10-04 09:50:14.951385925 +0200
@@ -2666,7 +2666,7 @@
     int         rv = 0;
 
     conn_ref = swig_make_ref(conn, ipmi_sol_conn_t);
-    swig_call_cb_rv('i', &rv, cb, "sol_data_received", "%p%*b",
+    swig_call_cb_rv('i', &rv, cb, "sol_data_received", "%p%*s",
                    &conn_ref, count, buf);
     swig_free_ref_check(conn_ref, ipmi_sol_conn_t);
     return rv;


------------------------------------------------------------------------------
CenturyLink Cloud: The Leader in Enterprise Cloud Services.
Learn Why More Businesses Are Choosing CenturyLink Cloud For
Critical Workloads, Development Environments & Everything In Between.
Get a Quote or Start a Free Trial Today. 
http://pubads.g.doubleclick.net/gampad/clk?id=119420431&iu=/4140/ostg.clktrk
_______________________________________________
Openipmi-developer mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openipmi-developer

Reply via email to