Date: Mon, 12 May 2003 06:30:27 +0300
From: "Eli Zaretskii" <eliz AT elta DOT co DOT il>
Sender: halo1 AT zahav DOT net DOT il
To: sandmann AT clio DOT rice DOT edu
Message-Id: <9743-Mon12May2003063026+0300-eliz@elta.co.il>
X-Mailer: emacs 21.3.50 (via feedmail 8 I) and Blat ver 1.8.9
CC: djgpp-workers AT delorie DOT com, dborca AT yahoo DOT com
In-reply-to: <10305112241.AA23271@clio.rice.edu> (sandmann@clio.rice.edu)
Subject: Re: DXE3 in DJGPP CVS breaks cross-compiling
References:  <10305112241 DOT AA23271 AT clio DOT rice DOT edu>
Reply-To: djgpp-workers AT delorie DOT com
Errors-To: nobody AT delorie DOT com
X-Mailing-List: djgpp-workers AT delorie DOT com
X-Unsubscribes-To: listserv AT delorie DOT com
Precedence: bulk

> From: sandmann AT clio DOT rice DOT edu (Charles Sandmann)
> Date: Sun, 11 May 2003 17:41:26 -0500 (CDT)
> 
> > To remedy this, we need to quote every arg that needs it (or maybe
> > just quote all of them).
> 
> I'd prefer not.  The old code didn't.

The old code used spawn* which doesn't require such characters to be
quoted.  spawn* functions already know that each arcg they get is a
single string, so any special characters there don't matter.

By contrast, `system' treats its argument as a blank-separated list
of strings, so quoting _is_ necessary.

> If we really require that, then maybe we don't need to have cross
> compile support and should leave it the way it is.

An alternative would be to rewrite the portability layer for spawn*
that uses exec*, not `system'.  exec* and spawn* share the wayt hey
treat their arguments: they both don't require quoting.