From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Brownell Date: Thu, 21 Nov 2002 04:32:06 +0000 Subject: Re: usb.usermap script processing doesn't work for me 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 Reinhold May wrote: > Hi, Welcome! > That didn't work for me and I tracked my problem down to an entry > in '/etc/hotplug/hotplug.functions' and changed it according to > this: > > @@ -119,7 +119,7 @@ > fi > > # do it with just shell scripts > - if [ "$LISTER" = "" ]; then > + if [ "$DRIVERS" = "" ]; then > ${TYPE}_map_modules < $FILENAME > fi > > As '$LISTER' is the command to check for corresponding kernel > modules and '${TYPE}_map_modules < $FILENAME' would save the > information from e. g. 'usb.usermap', it doesn't make any sense > to me. Checking if '$DRIVERS' has already been set by '$LISTER' > commands above that position does make far more sense to me and > in my case it worked. Actually the expectation is that the binary lister and the shell script version are _identical_ except that the binary one can recognize multi-interface devices ... so it would notice $DRIVERS that the shell script wouldn't, never the other way around. Seems that's not the case ... like maybe "usbmodules" doesn't understand about lines with comments, that sounds familiar. I'll merge your patch in with some other things I've got queued, mostly for some intelligence about 2.5 hotplug. > I hope you agree to me that this was not the intended behaviour > of the script and you agree it has to be changed. The intended behavior is that it works and creates more happy penguins! Merging your patch creates two more. World Domination, step by step ... :) - Dave ------------------------------------------------------- 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