[Nut-upsdev] [nut-commits] svn commit r1734 - in trunk/scripts: hal hotplug udev
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
> 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
I've logged a tracker as a reminder:
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.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the Nut-upsdev