From mboxrd@z Thu Jan 1 00:00:00 1970 From: Greg KH Date: Tue, 08 Feb 2005 20:15:36 +0000 Subject: Re: patches from the debian package Message-Id: <20050208201536.GA25328@kroah.com> List-Id: References: <20050205155031.GA12611@wonderland.linux.it> In-Reply-To: <20050205155031.GA12611@wonderland.linux.it> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: linux-hotplug@vger.kernel.org On Tue, Feb 08, 2005 at 08:52:29PM +0100, Kay Sievers wrote: > [offlist] Hm, looks like it was on the list to me :) > On Tue, 2005-02-08 at 20:50 +0100, Kay Sievers wrote: > > On Mon, 2005-02-07 at 10:39 -0800, Patrick Mansfield wrote: > > > On Mon, Feb 07, 2005 at 12:13:09AM +0100, Kay Sievers wrote: > > > > On Sun, 2005-02-06 at 11:05 -0800, Patrick Mansfield wrote: > > > > > Hmm, what about letting udev create the temporary node, instead of doing > > > > it in the *_id programs itself? That may be the best solution. > > > > > > > > Something like: > > > > PROGRAM="/sbin/scsi_id -d %N", ... > > > > > > > > What do you think? > > > > > > Yes, that would be nice. > > > > > > scsi_id pre-dates the addition of any of the % code, and relies on the > > > DEVPATH environment variable. > > > > > > So it also needs code changes to use the DEVPATH plus any command line > > > option, and/or a new % value for the sysfs path, like: > > > > > > PROGRAM="scsi_id -d %N -s %p", ... > > > > Ok, I did that. We have %N and %p now. > > You can pull that into your tree: > bk pull bk://vrfy.bkbits.net/udev > > if you are faster than Greg. :) I've pulled this into my tree and pushed it out now. thanks, greg k-h ------------------------------------------------------- SF email is sponsored by - The IT Product Guide Read honest & candid reviews on hundreds of IT Products from real users. Discover which products truly live up to the hype. Start reading now. http://ads.osdn.com/?ad_ide95&alloc_id396&op=click _______________________________________________ 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