From mboxrd@z Thu Jan 1 00:00:00 1970 From: Greg KH Date: Wed, 13 Oct 2004 23:56:16 +0000 Subject: Re: Problem parsing %s in udev rules Message-Id: <20041013235616.GA7427@kroah.com> List-Id: References: <1097105584.9273.9.camel@bluto.andrew> In-Reply-To: <1097105584.9273.9.camel@bluto.andrew> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: linux-hotplug@vger.kernel.org On Wed, Oct 13, 2004 at 05:13:12PM -0600, Andrew Patterson wrote: > > We already trim it off when matching, but we also allow matching if you > > do put the spaces in there. This patch breaks that, right? > > Correct, I have a new patch that trims after the comparison, so it > should work in both cases. Thanks, I've applied this. > > > I also wonder if it might be a good idea to do some sort of > > > transformation for characters in an attribute that are illegal in a file > > > system, rather than just chopping everything off. e.g, spaces that are > > > not leading or trailing would be changed to underscores. > > > > ick, no. > > Really? Care to elaborate? > > Do you not like any transformation (besides the "let's chop off > everything after anything illegal") or just the underscores for spaces > one? Perhaps a configuration value to make it optional? This sort of > thing might relieve a lot of the script writing that must be done to > handle cases that may be common. Have you run into any cases where something like this is needed? I don't really want to do any transformation within udev itself if possible, as it's a slippery slope down... Helper programs can be used if people run into anything really nasty, and the fact that this hasn't come up yet, lends me to belive it really isn't a problem. Do you have any device that would need such transformation? thanks, greg k-h ------------------------------------------------------- This SF.net email is sponsored by: IT Product Guide on ITManagersJournal Use IT products in your business? Tell us what you think of them. Give us Your Opinions, Get Free ThinkGeek Gift Certificates! Click to find out more http://productguide.itmanagersjournal.com/guidepromo.tmpl _______________________________________________ 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