Date: Tue, 23 Mar 1999 06:03:05 -0500
Message-Id: <199903231103.GAA18733@mescaline.gnu.org>
From: Eli Zaretskii <eliz AT gnu DOT org>
To: djgpp-workers AT delorie DOT com
In-reply-to: <199903211216.NAA04642@father.ludd.luth.se> (message from Martin
	Str|mberg on Sun, 21 Mar 1999 13:16:21 +0100 (MET))
Subject: Re: More TMPDIR strangeness
References:  <199903211216 DOT NAA04642 AT father DOT ludd DOT luth DOT se>
Reply-To: djgpp-workers AT delorie DOT com
X-Mailing-List: djgpp-workers AT delorie DOT com
X-Unsubscribes-To: listserv AT delorie DOT com
Precedence: bulk

   From: Martin Str|mberg <ams AT ludd DOT luth DOT se>

   --- gnu/emacs/src/callproc.c~   Mon Aug  5 01:15:02 1996
   +++ gnu/emacs/src/callproc.c    Sun Mar 21 13:05:10 1999
   @@ -333,7 +333,7 @@
      }

    #ifdef MSDOS /* MW, July 1993 */
   -  if ((outf = egetenv ("TMP")) || (outf = egetenv ("TEMP")))
   +  if ((outf = egetenv ("TMPDIR")) || (outf = egetenv ("TMP")) || (outf = egetenv ("TEMP")))
	strcpy (tempfile = alloca (strlen (outf) + 20), outf);
      else
	{

Thanks.  The current Emacs sources already have a similar change, but
it also makes sure at startup that a usable temporary directory
exists.  With your change, if none of the environment variables is
defined, the code will crash.

Also, I don't really recommend using a RAM disk for these purposes,
since call-process could result in really large files.