On Mon, 2015-06-08 at 09:45 +0200, Arnaud Quette wrote: > > > 2015-06-04 13:04 GMT+02:00 Bastien Nocera <had...@hadess.net>: > > On Sat, 2015-05-30 at 18:51 +0200, Arnaud Quette wrote: > > > > > > > > > 2015-05-29 14:09 GMT+02:00 Bastien Nocera <had...@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? > > > > Sure. > > > > > > 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. > > > > Just changing the path to project page URL + the path would be fine > > (that means that somebody unfamiliar with NUT's upstream can > > actually > > find the scripts to update this file). > > > Prior to pushing the changes in NUT and to generate the patches for > UPower, does the below header suit you: > > # Uninterruptible Power Supplies with USB HID interfaces > # > # http://cgit.freedesktop.org/upower/tree/rules/95-upower-hid.rules
That bit isn't necessary. > # > # This file was automatically generated by NUT: > # https://github.com/networkupstools/nut/ > # > # To keep up to date, monitor upstream NUT > # https://github.com/networkupstools/nut/commits/master/scripts/upower/95- > upower-hid.rules > # or checkout the NUT repository and call 'tools/nut-usbinfo.pl' > > Or do you see any other better wording? The rest is fine. Cheers _______________________________________________ devkit-devel mailing list devkit-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/devkit-devel