From mboxrd@z Thu Jan 1 00:00:00 1970 From: Patrick Mansfield Date: Mon, 07 Feb 2005 18:39:14 +0000 Subject: Re: patches from the debian package Message-Id: <20050207183914.GA22971@us.ibm.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 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", ... 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). -- Patrick Mansfield ------------------------------------------------------- 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