[Nut-upsdev] [nut-commits] svn commit r1734 - in trunk/scripts: hal hotplug udev

Arnaud Quette aquette.dev at gmail.com
Tue Jan 20 21:19:37 UTC 2009


2009/1/16 Arjen de Korte <nut+devel at de-korte.org <nut%2Bdevel at de-korte.org>>

> Citeren Arnaud Quette <aquette.dev at gmail.com>:
>
> > about the filtering, I thought Arjen already applied the 2nd macro over
> > USB_DEVICE...
>
> Not yet, since I don't think we already reached a consensus on which
> devices should go into which script. Basically, there are three
> categories:
>
> 1) valid VID:PID that uniquely matches a UPS
> 2) valid VID:PID that uniquely matches a USB to serial converter
> 3) bogus VID:PID
>
> Only for #1 there is no discussion that we can safely put this in the
> hotplug/udev/HAL scripts.
>
> Regarding #2 and #3, it is questionable since a device showing up with
> this VID:PID combination *might* be a UPS, but it could also be
> something totally different. I know of at least one non-UPS device (a
> GPS) that uses a VID:PID combination that is in category #2. I don't
> think we should use these in the HAL script and I'm uncertain if it is
> a good idea to use it in the hotplug/udev scripts. Putting them in
> risks that we're setting the device up for NUT (while it is something
> else) and not putting them in will probably lead to many questions on
> the mailinglist why device X worked in nut-2.2.2 and no longer works
> for nut-2.4.0.
>

thanks for the clarification on #2 since until now, I was only aware of #1
and #3

I've logged a tracker as a reminder:
https://alioth.debian.org/tracker/index.php?func=detail&aid=311356&group_id=30602&atid=411542

what comes first to my mind is:
- as you stated, remove #3 from all these helpers. NUT support should only
be kept through forcing the regex matching. This leaves the permissions
issue... see below
- manage permissions more globally within a group, as for serial devices
(dialout, uucp, ...)
This solution would also solve the #2 issue!
USB groups already exists, like usbusers on Debian. But as for serial devs,
this opens a wider door.
so we'll have to weight the pros and cons and make a decision here.

Arnaud
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.alioth.debian.org/pipermail/nut-upsdev/attachments/20090120/738776e2/attachment.htm 


More information about the Nut-upsdev mailing list