Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 16 Apr 2004 12:35:44 +0100 (BST)
From:      Matthew Seaman <m.seaman@infracaninophile.co.uk>
To:        FreeBSD-gnats-submit@FreeBSD.org
Subject:   docs/65615: make.conf.5 documents non-existent MAKE_SHELL variable
Message-ID:  <200404161135.i3GBZitP006602@happy-idiot-talk.infracaninophile.co.uk>
Resent-Message-ID: <200404161140.i3GBe2wI061436@freefall.freebsd.org>

next in thread | raw e-mail | index | archive | help

>Number:         65615
>Category:       docs
>Synopsis:       make.conf.5 documents non-existent MAKE_SHELL variable
>Confidential:   no
>Severity:       non-critical
>Priority:       medium
>Responsible:    freebsd-doc
>State:          open
>Quarter:        
>Keywords:       
>Date-Required:
>Class:          doc-bug
>Submitter-Id:   current-users
>Arrival-Date:   Fri Apr 16 04:40:02 PDT 2004
>Closed-Date:
>Last-Modified:
>Originator:     Matthew Seaman
>Release:        FreeBSD 4.10-BETA i386
>Organization:
Infracaninophile
>Environment:
System: FreeBSD happy-idiot-talk.infracaninophile.co.uk 4.10-BETA FreeBSD 4.10-BETA #65: Thu Apr 15 21:47:54 BST 2004 root@happy-idiot-talk.infracaninophile.co.uk:/usr/obj/usr/src/sys/HAPPY-IDIOT-TALK i386


>Description:

make.conf(5) describes the MAKE_SHELL variable:

     MAKE_SHELL    (str) Controls the shell used internally by make(1) to
                   process the command scripts in makefiles.  sh(1), ksh(1),
                   and csh(1) all currently supported.

but there appears to be no support anywhere in the sources for the use
of that variable in Makefiles:

    % find /usr/src -type f -print | xargs grep MAKE_SHELL 
    /usr/src/contrib/libreadline/doc/Makefile.in:SHELL = @MAKE_SHELL@
    /usr/src/contrib/libreadline/CHANGELOG: - set MAKE_SHELL to /bin/sh and substitute into the Makefiles
    /usr/src/contrib/libreadline/CHANGELOG: - set SHELL from @MAKE_SHELL@
    /usr/src/contrib/libreadline/Makefile.in:SHELL = @MAKE_SHELL@
    /usr/src/contrib/libreadline/configure:MAKE_SHELL=/bin/sh
    /usr/src/contrib/libreadline/configure:s%@MAKE_SHELL@%$MAKE_SHELL%g
    /usr/src/contrib/libreadline/configure.in:MAKE_SHELL=/bin/sh
    /usr/src/contrib/libreadline/configure.in:AC_SUBST(MAKE_SHELL)
    /usr/src/contrib/libreadline/examples/Makefile.in:SHELL = @MAKE_SHELL@
    /usr/src/contrib/libreadline/shlib/Makefile.in:SHELL = @MAKE_SHELL@
    /usr/src/share/man/man5/make.conf.5:.It Va MAKE_SHELL
    /usr/src/share/man/man5/make.conf.5:MAKE_SHELL?=sh

This functionality appears to be provided by the .SHELL pseudo target
built into make(1).


>How-To-Repeat:

>Fix:

--- make.conf.5.diff begins here ---
--- make.conf.5.orig	Fri Apr 16 11:49:20 2004
+++ make.conf.5	Fri Apr 16 11:49:59 2004
@@ -182,19 +182,6 @@
 make's in
 .Pa /usr/src
 in this variable.
-.It Va MAKE_SHELL
-.Pq Vt str
-Controls the shell used internally by
-.Xr make 1
-to process the command scripts in makefiles.
-.Xr sh 1 ,
-.Xr ksh 1 ,
-and
-.Xr csh 1
-all currently supported.
-.Bd -literal -offset indent
-MAKE_SHELL?=sh
-.Ed
 .It Va MTREE_FOLLOWS_SYMLINKS
 .Pq Vt str
 Set this to
--- make.conf.5.diff ends here ---


>Release-Note:
>Audit-Trail:
>Unformatted:



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200404161135.i3GBZitP006602>