Mailing-List: contact cygwin-help@cygwin.com; run by ezmlm
List-Subscribe: <mailto:cygwin-subscribe@cygwin.com>
List-Archive: <http://sources.redhat.com/ml/cygwin/>
List-Post: <mailto:cygwin@cygwin.com>
List-Help: <mailto:cygwin-help@cygwin.com>, <http://sources.redhat.com/ml/#faqs>
Sender: cygwin-owner@cygwin.com
Mail-Followup-To: cygwin@cygwin.com
Delivered-To: mailing list cygwin@cygwin.com
Message-ID: <3E2C7C08.5030005@ece.gatech.edu>
Date: Mon, 20 Jan 2003 17:45:28 -0500
From: Charles Wilson <cwilson@ece.gatech.edu>
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.0.1) Gecko/20020823 Netscape/7.0
X-Accept-Language: en-us, en
MIME-Version: 1.0
To: Bruce Korb <bkorb@gnu.org>
CC: Earnie Boyd <earnie_boyd@yahoo.com>, automake-patches@gnu.org,
   libtool-patches@gnu.org, mingw-users@lists.sourceforge.net, libtool@gnu.org,
   Alexandre Duret-Lutz <duret_g@lrde.epita.fr>, cygwin@cygwin.com
Subject: Re: [Mingw-users] Re: Solving the "relink exe's" libtool problem[take
 3]
References: <3E19C657.1040904@ece.gatech.edu>		<2003-01-09-17-11-09+16471+duret_g@lrde.epita.fr>		<3E1DE146.8030901@ece.gatech.edu>		<2003-01-09-23-01-51+2104+duret_g@lrde.epita.fr>		<3E20BF9A.4080400@ece.gatech.edu>		<2003-01-13-00-19-29+30521+duret_g@lrde.epita.fr>		<3E224479.8030604@ece.gatech.edu> <3E227106.5000706@ece.gatech.edu> <3E2C4478.3040507@yahoo.com> <3E2C4E93.144C5B40@gnu.org>
Content-Type: text/plain; charset=us-ascii; format=flowed
Content-Transfer-Encoding: 7bit

Bruce Korb wrote:
> Earnie Boyd wrote:
> 
>>This patch passes my test.  What do we need to do to get this accepted
>>into libtool cvs HEAD?
> 
> 
>>>+  newargz[0] = xstrdup("/bin/sh");
>>
> 
> This may not be the shell and there is no point allocating it.
> It is fine to use it from static memory.

Okay, the second comment (use static string, not allocated memory) is 
easy enough.  But what's the best way to use "the shell"?  Do a unquoted 
replacement (<<EOF, not <<"EOF") e.g.

   ...
   newargz = XMALLOC(char *, argc+2);
EOF
$echo >> $cwrappersource <<EOF
   newargz[0] = \"$SHELL\";
EOF
$echo >> $cwrappersource <<"EOF"
   newargz[1] = fnqualify(argv[0]);
   ...

?

--Chuck



--
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple
Bug reporting:         http://cygwin.com/bugs.html
Documentation:         http://cygwin.com/docs.html
FAQ:                   http://cygwin.com/faq/