delorie.com/archives/browse.cgi | search |
Charles Wilson wrote: > > 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 I wouldn't have mentioned the static string by itself ;-) > easy enough. But what's the best way to use "the shell"? Do a unquoted > replacement (<<EOF, not <<"EOF") e.g. Yes. Somewhere, buried in the configury stuff is an environment variable named something like, "CONFIG_SHELL". That's what you want. If it is not available, then imitating the techniques used to obtain it by configure should be used. -- 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/
webmaster | delorie software privacy |
Copyright © 2019 by DJ Delorie | Updated Jul 2019 |