<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">2015-06-04 13:04 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:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="">On Sat, 2015-05-30 at 18:51 +0200, Arnaud Quette wrote:<br>
><br>
><br>
> 2015-05-29 14:09 GMT+02:00 Bastien Nocera <<a href="mailto:hadess@hadess.net">hadess@hadess.net</a>>:<br>
> > 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<br>
> > and 2<br>
> > > Liebert)<br>
> ><br>
> > Could you please split those changes into 3 separate patches?<br>
><br>
> since you already have the "usbmisc", I can possibly check to split<br>
> in 2 commits (1 for the comments, and 1 for the content update).<br>
> would that suits you that way?<br>
<br>
</span>Sure.<br>
<span class=""><br>
> >  It would also be useful to include a full URL to the NUT Perl<br>
> > script<br>
> > (to a git repository perhaps), so people don't need to check out<br>
> > the<br>
> > full repo to update it.<br>
> ><br>
> as per Charles comments, beside from the perl script, you need all<br>
> the drivers/*hid.c files to extract the USB info.<br>
> the added comment was just to shed light on the fact that it's an<br>
> automated data extraction.<br>
> I can reword to make it more clear if you want.<br>
<br>
</span>Just changing the path to project page URL + the path would be fine<br>
(that means that somebody unfamiliar with NUT's upstream can actually<br>
find the scripts to update this file).<br>
</blockquote></div><br></div><div class="gmail_extra">Prior to pushing the changes in NUT and to generate the patches for UPower, does the below header suit you:<br><br># Uninterruptible Power Supplies with USB HID interfaces<br>#<br># <a href="http://cgit.freedesktop.org/upower/tree/rules/95-upower-hid.rules">http://cgit.freedesktop.org/upower/tree/rules/95-upower-hid.rules</a><br>#<br># This file was automatically generated by NUT:<br># <a href="https://github.com/networkupstools/nut/">https://github.com/networkupstools/nut/</a><br># <br># To keep up to date, monitor upstream NUT<br># <a href="https://github.com/networkupstools/nut/commits/master/scripts/upower/95-upower-hid.rules">https://github.com/networkupstools/nut/commits/master/scripts/upower/95-upower-hid.rules</a><br># or checkout the NUT repository and call 'tools/<a href="http://nut-usbinfo.pl">nut-usbinfo.pl</a>'<br><br></div><div class="gmail_extra">Or do you see any other better wording?<br clear="all"></div><div class="gmail_extra"><br></div><div class="gmail_extra">thanks and cheers,<br></div><div class="gmail_extra">Arno<br></div><div class="gmail_extra">-- <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>