[PATCH] scsi_id temporary directory

[PATCH] scsi_id temporary directory

[Marco d'Itri]

[-- Attachment #1.1: Type: text/plain, Size: 148 bytes --]

/tmp is not suitable as a temporary directory because it's usually
mounted read only at the time udevstart runs the program.

-- 
ciao,
Marco

[-- Attachment #1.2: scsi_id_tmp --]
[-- Type: text/plain, Size: 498 bytes --]

diff -ruNp udev-050.orig/extras/scsi_id/scsi_id.c udev-050/extras/scsi_id/scsi_id.c
--- udev-050.orig/extras/scsi_id/scsi_id.c	2004-12-18 06:53:07.000000000 +0100
+++ udev-050/extras/scsi_id/scsi_id.c	2005-01-01 22:19:15.000000000 +0100
@@ -44,8 +44,8 @@
 /*
  * temporary names for mknod.
  */
-#define TMP_DIR	"/tmp"
-#define TMP_PREFIX "scsi"
+#define TMP_DIR	"/dev"
+#define TMP_PREFIX "tmp-scsi"
 
 /*
  * XXX Note the 'e' (send output to stderr in all cases), and 'c' (callout)

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 189 bytes --]

Re: [PATCH] scsi_id temporary directory

[Patrick Mansfield]

On Tue, Jan 04, 2005 at 09:58:42PM +0100, Marco d'Itri wrote:
> /tmp is not suitable as a temporary directory because it's usually
> mounted read only at the time udevstart runs the program.

OK ...

Do you think it should have an option like "-t directory", defaulting to
/dev?

-- Patrick Mansfield


-------------------------------------------------------
The SF.Net email is sponsored by: Beat the post-holiday blues
Get a FREE limited edition SourceForge.net t-shirt from ThinkGeek.
It's fun and FREE -- well, almost....http://www.thinkgeek.com/sfshirt
_______________________________________________
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