<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">2015-05-29 14:09 GMT+02:00 Bastien Nocera <span dir="ltr"><<a href="mailto:hadess@hadess.net" target="_blank">hadess@hadess.net</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Fri, 2015-05-29 at 13:59 +0200, Arnaud Quette wrote:<br>
> Hi Richard and the list,<br>
><br>
> you'll find attached a patch for 95-upower-hid.rules, which adds:<br>
> - the "usbmisc" filtering, as added in your repo,<br>
> - more comments, including one that points at your UPower repo,<br>
> - 3 new manufacturers (Minibox, iDowell and Powerware)<br>
> - a bunch of new devices (7 HP,  1 APC, 1 TrippLite, 2 PowerCOM and 2<br>
> Liebert)<br>
<br>
</span>Could you please split those changes into 3 separate patches?<br></blockquote><div> <br></div><div>since you already have the "usbmisc", I can possibly check to split in 2 commits (1 for the comments, and 1 for the content update).<br></div><div>would that suits you that way?<br></div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
It would also be useful to include a full URL to the NUT Perl script<br>
(to a git repository perhaps), so people don't need to check out the<br>
full repo to update it.<br>
</blockquote></div><br></div><div class="gmail_extra">as per Charles comments, beside from the perl script, you need all the drivers/*hid.c files to extract the USB info.<br></div><div class="gmail_extra">the added comment was just to shed light on the fact that it's an automated data extraction.<br></div><div class="gmail_extra">I can reword to make it more clear if you want.<br><br></div><div class="gmail_extra">cheers,<br></div><div class="gmail_extra">Arno<br>-- <br><div class="gmail_signature"><div dir="ltr">Eaton Data Center Automation - Opensource Leader<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></div></div>
</div></div>