<div dir="ltr">Hello Bastien,<div class="gmail_extra"><br><div class="gmail_quote">2015-06-08 15:07 GMT+02:00 Arnaud Quette <span dir="ltr"><<a href="mailto:arnaud.quette@gmail.com" target="_blank">arnaud.quette@gmail.com</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"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><div><div class="h5">2015-06-08 13:06 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"><div><div>On Mon, 2015-06-08 at 09:45 +0200, Arnaud Quette wrote:<br>
><br>
><br>
> 2015-06-04 13:04 GMT+02:00 Bastien Nocera <<a href="mailto:hadess@hadess.net" target="_blank">hadess@hadess.net</a>>:<br>
> > 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" target="_blank">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<br>
> > adds:<br>
> > > > > - the "usbmisc" filtering, as added in your repo,<br>
> > > > > - more comments, including one that points at your UPower<br>
> > repo,<br>
> > > > > - 3 new manufacturers (Minibox, iDowell and Powerware)<br>
> > > > > - a bunch of new devices (7 HP,  1 APC, 1 TrippLite, 2<br>
> > 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<br>
> > split<br>
> > > in 2 commits (1 for the comments, and 1 for the content update).<br>
> > > would that suits you that way?<br>
> ><br>
> > Sure.<br>
> ><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<br>
> > out<br>
> > > > the<br>
> > > > full repo to update it.<br>
> > > ><br>
> > > as per Charles comments, beside from the perl script, you need<br>
> > 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>
> > Just changing the path to project page URL + the path would be fine<br>
> > (that means that somebody unfamiliar with NUT's upstream can<br>
> > actually<br>
> > find the scripts to update this file).<br>
> ><br>
> Prior to pushing the changes in NUT and to generate the patches for<br>
> 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" target="_blank">http://cgit.freedesktop.org/upower/tree/rules/95-upower-hid.rules</a><br>
<br>
</div></div>That bit isn't necessary.<span><br></span></blockquote><div><br></div></div></div><div>if you don't mind, I would prefer to keep it somewhere, just to remember (from a NUT point of view) where the upower repos. (and that file) is.<br></div><div>so, not necessary, but useful for the NUT side...<br></div><span class=""><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span>
> #<br>
> # This file was automatically generated by NUT:<br>
> # <a href="https://github.com/networkupstools/nut/" target="_blank">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-" target="_blank">https://github.com/networkupstools/nut/commits/master/scripts/upower/95-</a><br>
> upower-hid.rules<br>
> # or checkout the NUT repository and call 'tools/<a href="http://nut-usbinfo.pl" target="_blank">nut-usbinfo.pl</a>'<br>
><br>
> Or do you see any other better wording?<br>
<br>
</span>The rest is fine.<br></blockquote><div> <br></div></span><div>ok, thanks.<br><br></div><div>I'll just wait for your ack on the above UPower repos. point, to push the remaining commit.<br></div></div></div></div></blockquote></div><br></div><div class="gmail_extra">here is the patch, before I completly forgot.<br></div><div class="gmail_extra">Not that I've removed the pointer to UPower git repository, as you wanted.<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>