[Nut-upsdev] UPower: 95-upower-hid.rules update

Arnaud Quette arnaud.quette at gmail.com
Sat May 30 16:51:01 UTC 2015


2015-05-29 14:09 GMT+02:00 Bastien Nocera <hadess at hadess.net>:

> On Fri, 2015-05-29 at 13:59 +0200, Arnaud Quette wrote:
> > Hi Richard and the list,
> >
> > you'll find attached a patch for 95-upower-hid.rules, which adds:
> > - the "usbmisc" filtering, as added in your repo,
> > - more comments, including one that points at your UPower repo,
> > - 3 new manufacturers (Minibox, iDowell and Powerware)
> > - a bunch of new devices (7 HP,  1 APC, 1 TrippLite, 2 PowerCOM and 2
> > Liebert)
>
> Could you please split those changes into 3 separate patches?
>

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).
would that suits you that way?

It would also be useful to include a full URL to the NUT Perl script
> (to a git repository perhaps), so people don't need to check out the
> full repo to update it.
>

as per Charles comments, beside from the perl script, you need all the
drivers/*hid.c files to extract the USB info.
the added comment was just to shed light on the fact that it's an automated
data extraction.
I can reword to make it more clear if you want.

cheers,
Arno
-- 
Eaton Data Center Automation - Opensource Leader
NUT (Network UPS Tools) Project Leader - http://www.networkupstools.org
Debian Developer - http://www.debian.org
Free Software Developer - http://arnaud.quette.fr
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.alioth.debian.org/pipermail/nut-upsdev/attachments/20150530/14d0711a/attachment.html>


More information about the Nut-upsdev mailing list