<br><div class="gmail_quote">2012/11/19 Elio Parisi <span dir="ltr"><<a href="mailto:E.Parisi@riello-ups.com" target="_blank">E.Parisi@riello-ups.com</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">




<div link="blue" vlink="purple" lang="IT">
<p></p><div><p class="MsoNormal">Hi Arnaud, hi developers,<u></u><u></u></p><p class="MsoNormal"><u></u></p></div></div></blockquote><div><br>Hi Elio and Massimo,<br> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div link="blue" vlink="purple" lang="IT"><div><p class="MsoNormal"></p><p class="MsoNormal"><span lang="EN-US">attach there is the patch for riello drivers.<u></u><u></u></span></p><p class="MsoNormal"><span lang="EN-US"><u></u> <u></u></span></p>

<p class="MsoNormal"><span lang="EN-US">Can you tell me if there is other to change in order to accomplish your rules and<u></u><u></u></span></p><p class="MsoNormal"><span lang="EN-US">if the steps I have followed to create the patch are all rights?<u></u><u></u></span></p>

<p class="MsoNormal"><span lang="EN-US"><u></u></span></p></div></div></blockquote><div><br>thanks for your efforts, and congratulations: your drivers are now part of the NUT mainline:<br><a href="http://trac.networkupstools.org/projects/nut/changeset/3784">http://trac.networkupstools.org/projects/nut/changeset/3784</a><br>
<br>Note that I've had to fix a few things:<br>
- missing references to the new manpages in docs/man/Makefile.am: your manpages were not generated<br>- missing reference to riello_usb in tools/<a href="http://nut-usbinfo.pl">nut-usbinfo.pl</a>: it's needed to generated udev files, to set device permissions on Linux<br>

- out-of-date call to is_usb_device_supported(): the function prototype has changed in Subversion r3721 (2012-09-10)<br>- I improved the USB_DEVICE declaration in riello_usb.c: I've added a define for Riello, and replaced the comment (Agiler) with something more suitable. These comments are used in various places, like udev files.<br>

- I've fixed some remaining c++ style comments (riello.c)<br>- I've removed extraneous spaces, tabs and some samples of code that were not useful anymore<br>- I've <span id="result_box" class="short_text" lang="en"><span class="hps">condensed entries in the driver.list</span></span><br>
<br>As a side note, it's nice to see that Massimo participated, as manpage author ;)<br>
<br>There are still some things that I would appreciate you fix (easier for maintenance), such as:<br>- english variables names for: zapisuj, buf_ptr_spracuj_port, cakaj_data, ...<br>- english functions name for: riello_koniec(), riello_prislo_nak(), riello_spracuj_port()<br>

<br>I would also like to see some evidences of working devices, and some details on the testing you've done with these drivers.<br>Sample upsc outputs would be also appreciated for the <a href="http://www.networkupstools.org/devdumps/">Devices Dumps Library</a>.<br>
<br>Finally, please subscribe to the NUT users mailing list for supporting Riello users:<br><a href="http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/nut-upsuser">http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/nut-upsuser</a><br>
<br>Thanks for your contribution.<br><br>cheers,<br clear="all"></div></div>Arnaud<br>-- <br>NUT (Network UPS Tools) 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.fr" target="_blank">http://arnaud.quette.fr</a><br><br>