Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 1 Dec 2005 14:38:11 -0500
From:      Garance A Drosehn <gad@FreeBSD.org>
To:        Ruslan Ermilov <ru@FreeBSD.org>, Warner Losh <imp@FreeBSD.org>
Cc:        cvs-src@FreeBSD.org, src-committers@FreeBSD.org, cvs-all@FreeBSD.org
Subject:   Re: cvs commit: src Makefile.inc1
Message-ID:  <p06230900bfb4fa9993bf@[128.113.24.47]>
In-Reply-To: <20051201172757.GG20961@ip.net.ua>
References:  <200512011708.jB1H8u3Y065515@repoman.freebsd.org> <20051201172757.GG20961@ip.net.ua>

next in thread | previous in thread | raw e-mail | index | archive | help
At 7:27 PM +0200 12/1/05, Ruslan Ermilov wrote:
>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.
>
>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.

Hmm.  How about:

BUILDENVSHELL?=sh
buildenv:
	@echo Entering world for ${TARGET_ARCH}:${TARGET}
	@cd ${.CURDIR} && env ${WMAKEENV} ${BUILDENVSHELL} || true

-- 
Garance Alistair Drosehn     =      gad@gilead.netel.rpi.edu
Senior Systems Programmer               or   gad@FreeBSD.org
Rensselaer Polytechnic Institute;             Troy, NY;  USA



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