2009/1/16 Arjen de Korte <[email protected] <nut%[email protected]>>
> Citeren Arnaud Quette <[email protected]>: > > > 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
_______________________________________________ Nut-upsdev mailing list [email protected] http://lists.alioth.debian.org/mailman/listinfo/nut-upsdev
