<br><div class="gmail_quote">2012/1/9 Sadara Kael <span dir="ltr"><<a href="mailto:sadara@wildwoodgroup.com.au">sadara@wildwoodgroup.com.au</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
hi all,<br></blockquote><div><br>Hi Sadara,<br> </div><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
really simple fix for the bestups driver:<br>
<br>
In function:<br>
upsdrv_makevartable()<br>
<br>
you need to add the line<br>
addvar(VAR_VALUE, "battvoltmult", "Battery voltage multiplier");<br>
<br>
There are a number of simple improvements I could make to this driver, but I'm assuming that since the above bug has been around since nut v2.4, not many people are using this driver.<br></blockquote><div><br>Looking at the driver and your fix, this was indeed missing.<br>
Documentation (manpage) was present, but the addvar() call was missing.<br>I've just fixed it (trunk r3382), and bumped the driver version to 1.06.<br><br></div><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">

Is this driver worth improving?<br clear="all"></blockquote></div><br>As long as there is a user of the device / driver, there is an interest in improvement.<br>The rest is due to either a lack of developer(s), or knowledges from the users to qualify a bug and contribute a patch.<br>
<br>thanks for your contribution,<br>Arnaud<br>-- <br>Linux / Unix Expert R&D - Eaton - <a href="http://powerquality.eaton.com" target="_blank">http://powerquality.eaton.com</a><br>Network UPS Tools (NUT) Project Leader - <a href="http://www.networkupstools.org/" target="_blank">http://www.networkupstools.org/</a><br>
Debian Developer - <a href="http://www.debian.org" target="_blank">http://www.debian.org</a><br>Free Software Developer - <a href="http://arnaud.quette.free.fr/" target="_blank">http://arnaud.quette.free.fr/</a><br><br>