Re: Problem parsing %s in udev rules

[Greg KH <greg@kroah.com>]

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