cvs commit: src Makefile.inc1

Ruslan Ermilov ru at FreeBSD.ORG
Thu Dec 1 17:30:59 GMT 2005


On Thu, Dec 01, 2005 at 05:08:56PM +0000, Warner Losh wrote:
> imp         2005-12-01 17:08:56 UTC
> 
>   FreeBSD src repository
> 
>   Modified files:
>     .                    Makefile.inc1 
>   Log:
>   In make buildenv, spell sh as ${SHELL}.  In this case, we don't want
>   to hard code /bin/sh since we're forking it for the user to type
>   commands into.  As such, ${SHELL} is the preferred thing users type
>   commands into.
>   
>   Revision  Changes    Path
>   1.524     +1 -1      src/Makefile.inc1
> 
Please consider backing it out.  In this case, we want to hard code it
to "sh" since the stock ~root/.cshrc always resets $PATH, which breaks
"make buildenv" for root (which most of us use).  Testcase: "which cc"
from under "make buildenv" shell.


Cheers,
-- 
Ruslan Ermilov
ru at FreeBSD.org
FreeBSD committer
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 187 bytes
Desc: not available
Url : http://lists.freebsd.org/pipermail/cvs-all/attachments/20051201/b79d0943/attachment.bin


More information about the cvs-all mailing list