Date: Sun, 20 Aug 2000 17:39:20 +1000 (EST) From: Bruce Evans <bde@zeta.org.au> To: Brian Fundakowski Feldman <green@FreeBSD.org> Cc: "Andrey A. Chernov" <ache@nagual.pp.ru>, Marcel Moolenaar <marcel@cup.hp.com>, cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/usr.bin/make Makefile config.h job.c main.c Message-ID: <Pine.BSF.4.21.0008201718200.1042-100000@besplex.bde.org> In-Reply-To: <Pine.BSF.4.21.0008192335330.95612-100000@green.dyndns.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Sat, 19 Aug 2000, Brian Fundakowski Feldman wrote: > On Sun, 20 Aug 2000, Andrey A. Chernov wrote: > > > On Sun, Aug 20, 2000 at 07:55:20AM +1000, Bruce Evans wrote: > > > They seem to be remarkably broken. A POSIX.2 draft specifies that make > > > shall support the SHELL variable, but our make doesn't. (SHELL has the > > > [deleted] The "[deleted]" part was mostly wrong. The SHELL environment variable is required not to affect the shell used by `make'. Only definitions of SHELL in makefiles and on the command line affect the shell used by make. SHELL must be set to a default by `make'. > > > I'm not sure if .SHELL has any advantages over SHELL. Anyway, SHELL should > > > be fixed first since it is standard. The fix may be as simple as setting > > > > Thanks, Bruce, it is what I try to say from the very beginning of this > > thread. > > So we should make pmake support ${SHELL}? This would allow me to > change it on all compiles easily, but it would break things all over > the place because things set SHELL without expecting make(1) to be > using that shell. No problem, since make is not permitted to use these. There is a minor problem with implementing the default setting of SHELL in sys.mk. The current "SHELL ?= sh" usually gives the environment value. > The only functionality I really need is the ability > to override the default shell value. Even the variable ${SHELL} does > not provide that. I think .SHELL: path=ksh in /etc/make.conf.local is supposed to do this, but it currently gives core dumps for freeing of a bad pointer. Bruce To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?Pine.BSF.4.21.0008201718200.1042-100000>