From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Maksim (Max) Krasnyanskiy" Date: Mon, 22 Jul 2002 22:03:41 +0000 Subject: Re: Support for multiple usermaps in hotplug usb.agent Message-Id: List-Id: References: In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: linux-hotplug@vger.kernel.org At 09:55 PM 7/21/2002 -0700, David Brownell wrote: >Maksim (Max) Krasnyanskiy wrote: >>>Why do you rebuild the usermap database. Would it be easier to just find >>>a group of usermap's via a glob and then parse all of them each hotplug, >>>like Maksim's patch ? >>> >>>The only advantage I can think of rebuilding the database, is you can >>>better control conflicts, but that's a tricky situation anyway. >>They don't resolve any conflicts. update.usermap is pretty much just >> for map in $(all installed maps); do >> cat $map >> /etc/hotplug/usb.usermap >> done > >So at a technical level, ignoring the "where does stuff install" issue >(we want just one answer, the path doesn't much matter) the only real >difference is who does the match/catenate, and when. > >In fact, if Max's patch had scanned /usr/lib/hotplug/*/usb.usermap, >then there'd be compatibility with Debian, and Debian could get rid >of the "update.usermap" script. Yes ? I guess so. It should work for them. btw if we simply kept support for the central usb.usermap it will work fine with current update-usb.usermap. >How did /usr/lib/hotplug get settled on? There's also some code using >/usr/share/usb (/usr/lib is wrong for sharable stuff ...) for firmware >downloading, and I see that RedHat has /usr/share/hwdata/usb.ids ... that >is, not using /usr/share/usb either. Does the Linux Filesystem standard >have anything useful to say about this? Shouldn't it? > >In general I tend to think avoiding extra commands ("update.usermap") >is a good thing, but then so is avoiding wildcards in shell scripts. >In this case I'd prefer the latter, I must say. I don't see how wildcards would be a problem (assuming they're used appropriately). Anyway in this case update.usermap is extra commands and shell wildcards ;) (update.usermap uses shell wildcards). Max ------------------------------------------------------- This sf.net email is sponsored by:ThinkGeek Welcome to geek heaven. http://thinkgeek.com/sf _______________________________________________ Linux-hotplug-devel mailing list http://linux-hotplug.sourceforge.net Linux-hotplug-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/linux-hotplug-devel