From mboxrd@z Thu Jan 1 00:00:00 1970 From: Greg KH Date: Tue, 12 Oct 2004 23:58:18 +0000 Subject: Re: Problem parsing %s in udev rules Message-Id: <20041012235818.GB10872@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="iso-8859-1" Content-Transfer-Encoding: quoted-printable To: linux-hotplug@vger.kernel.org On Tue, Oct 12, 2004 at 05:04:21PM -0600, Andrew Patterson wrote: > Kay, >=20 > On Thu, 2004-10-07 at 02:30 +0200, Kay Sievers wrote: > > On Wed, Oct 06, 2004 at 06:23:05PM -0600, Andrew Patterson wrote: > > > On Thu, 2004-10-07 at 01:50 +0200, Kay Sievers wrote: > > > > On Wed, Oct 06, 2004 at 05:33:04PM -0600, Andrew Patterson wrote: > > > > > Udev seems to be chopping off anything seen in a rule after a %s.= For > > > > > example, if I have the following rule: > > > > >=20 > > > > > BUS=3D"scsi", KERNEL=3D"sd*", PROGRAM=3D"/sbin/scsi_id", NAME=3D"= %k", > > > > > SYMLINK=3D"scsi_disks/some-proceeding-stuff-%s{model}-some-follow= ing- > > > > > stuff" > > > > >=20 > > > > > Everything after the %s{model} is ignored. > > > > >=20 > > > > > Furthermore, everything in model following whitespace is ignored = as > > > > > well. I.e, if model contains "HP 34.7G", the resulting device fi= le > > > > > created is /dev/scsi_disks/some-proceeding-stuff-HP. > > > > >=20 > > > > > I used the following command to run this test: > > > > >=20 > > > > > # DEVPATH=3D/block/sda ACTION=ADd ./udev block > > > > >=20 > > > > > I also get the same behavior when using udevstart (patched to fix > > > > > parameter passing problems). > > > >=20 > > > > If your sysfs value contains spaces, which is the delimiter for mul= tiple > > > > symlinks, it will not work. > > > >=20 > > > > udev can't do this now, you need to wrap the call with an external = script, > > > > which ensures sane values. > > > >=20 > > >=20 > > > That explains the spaces. What about stuff trailing %s, if %s does n= ot > > > contain spaces. I.e, in the above example, model is ST336753LC and t= he > > > resulting device file is /dev/scsi_disks/some-proceding-stuff- > > > ST336753LC. > >=20 > > I expect the model value has trailing spaces. > >=20 > > You may look with: > > udevinfo -a -p /block/sdX >=20 > Yes it does, and it seems for most SCSI devices, vendor and model will > have trailing spaces. It all depends on the vendor and model :) > I have included a patch to udev-036 to deal with > this issue. It trims off trailing whitespace for all sysfs attributes. > It might be better to trim off leading whitespace as well. =20 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? > 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. 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