Mailing-List: contact cygwin-help AT cygwin DOT com; run by ezmlm
List-Subscribe: <mailto:cygwin-subscribe AT cygwin DOT com>
List-Archive: <http://sources.redhat.com/ml/cygwin/>
List-Post: <mailto:cygwin AT cygwin DOT com>
List-Help: <mailto:cygwin-help AT cygwin DOT com>, <http://sources.redhat.com/ml/#faqs>
Sender: cygwin-owner AT cygwin DOT com
Mail-Followup-To: cygwin AT cygwin DOT com
Delivered-To: mailing list cygwin AT cygwin DOT com
Message-ID: <3E2C7C08.5030005@ece.gatech.edu>
Date: Mon, 20 Jan 2003 17:45:28 -0500
From: Charles Wilson <cwilson AT ece DOT gatech DOT 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 AT gnu DOT org>
CC: Earnie Boyd <earnie_boyd AT yahoo DOT com>, automake-patches AT gnu DOT org,
   libtool-patches AT gnu DOT org, mingw-users AT lists DOT sourceforge DOT net, libtool AT gnu DOT org,
   Alexandre Duret-Lutz <duret_g AT lrde DOT epita DOT fr>, cygwin AT cygwin DOT com
Subject: Re: [Mingw-users] Re: Solving the "relink exe's" libtool problem[take
 3]
References: <3E19C657 DOT 1040904 AT ece DOT gatech DOT edu>		<2003-01-09-17-11-09+16471+duret_g AT lrde DOT epita DOT fr>		<3E1DE146 DOT 8030901 AT ece DOT gatech DOT edu>		<2003-01-09-23-01-51+2104+duret_g AT lrde DOT epita DOT fr>		<3E20BF9A DOT 4080400 AT ece DOT gatech DOT edu>		<2003-01-13-00-19-29+30521+duret_g AT lrde DOT epita DOT fr>		<3E224479 DOT 8030604 AT ece DOT gatech DOT edu> <3E227106 DOT 5000706 AT ece DOT gatech DOT edu> <3E2C4478 DOT 3040507 AT yahoo DOT com> <3E2C4E93 DOT 144C5B40 AT gnu DOT 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/