From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kay Sievers Date: Tue, 08 Feb 2005 19:50:10 +0000 Subject: Re: patches from the debian package Message-Id: <1107892210.5536.12.camel@localhost.localdomain> 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 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: > > > On Sat, Feb 05, 2005 at 04:50:31PM +0100, Marco d'Itri wrote: > > > > > > > > > > > scsi_id_tmp: when udevstart is started, /tmp is not writeable. > > > > > > > > > > What if /dev does not exist (udev_root modified) or perhaps not writable? > > > > > > Does udev put temp files under /dev? > > > > 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. > It should still figure out whether or not to print a trailing "\n" for > normal command line usage. DEVPATH set could be the trigger, though that > is not clean (using -s for sysfs path, but then using DEVPATH to > figure out if \n is printed). RESULT does not care about a trailing \n. Thanks, Kay ------------------------------------------------------- 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