On 2007.11.11 17:44:14 +0000, Johannes Schindelin wrote: > Hi, > > On Sun, 11 Nov 2007, Björn Steinbrink wrote: > > > The git wrapper executable always prepends the GIT_EXEC_PATH build > > variable to the current PATH, so prepending "." to the PATH is not > > enough to give precedence to the fake vi executable. > > > > The --exec-path option allows to prepend a directory to PATH even before > > GIT_EXEC_PATH (which is added anyway), so we can use that instead. > > Hmm. This will probably stop working when you do not have git installed, > because you now tell git to search for git programs in ".", where they are > not. Probably git-commit executes your installed write-tree, commit-tree > and friends, instead of the compiled ones. The . is prepended to PATH in _addition_ to the usual paths (as I wrote, see setup_path() in exec_cmd.c). It does not replace anything AFAICT. $ echo $PATH /home/doener/bin:/usr/local/bin:/usr/bin:/bin:/usr/games $ GIT_EXEC_PATH=.. GIT_EDITOR="env;" ../git commit | grep ^PATH= PATH=/home/doener/src/git:/home/doener/bin:/home/doener/src/git:/home/doener/bin:/usr/local/bin:/usr/bin:/bin:/usr/games $ GIT_EXEC_PATH=.. GIT_EDITOR="env;" ../git --exec-path=. commit | grep ^PATH= PATH=/home/doener/src/git/t:/home/doener/src/git:/home/doener/bin:/home/doener/src/git:/home/doener/bin:/usr/local/bin:/usr/bin:/bin:/usr/games Looks good to me... Björn - To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
