Re: [PATCH 2/2] Remember and use GIT_EXEC_PATH on exec()'s

From: Michal Ostrowski <mostrows@watson.ibm.com>
Date: Jan 11, 2006, 02:09 UTC
Message-ID: <1136945343.11717.638.camel@brick.watson.ibm.com>
In-Reply-To: <7vd5iz4mt7.fsf@assigned-by-dhcp.cox.net>

On Tue, 2006-01-10 at 16:42 -0800, Junio C Hamano wrote:

Show 10 quoted lines

> Michal Ostrowski <mostrows@watson.ibm.com> writes:
> 
> > How about searching for executables in the following places, and in this
> > order:
> >
> > 1. --exec-path setting, if any
> > 2. GIT_EXEC_PATH env var, if set
> > 3. PATH (never modified)
> > 4. Value of ${bindir} at build time
>

Show 11 quoted lines

> and then make the rule for git things:
> 
>  1. --exec-path
>  2. GIT_EXEC_PATH environment
>  3. $(gitexecdir)
> 
> in this order.  Non git things should just use $PATH without
> looking at anything else --- as long as a hook script calls a git
> wrapper (i.e. "git foo" not "git-foo") I think things should
> work fine.
>

The patch that follows implements your suggested search order and includes suggested Makefile changes.

-- 
Michal Ostrowski <mostrows@watson.ibm.com>