Propagate --exec-path setting to external commands via GIT_EXEC_PATH
authorJohannes Sixt <>
Sat, 21 Mar 2009 22:21:18 +0000 (23:21 +0100)
committerJunio C Hamano <>
Sun, 22 Mar 2009 04:45:14 +0000 (21:45 -0700)
Let PATH0=$PATH that was set before the invocation.
Let /foo be a build directory.
Let /pfx be the installation prefix.
Let pfxexecpath=/pfx/libexec/git-core.

The following is going on when 'git --exec-path=/foo gc' is invoked:

1. git sets PATH=/foo:$PATH0 using the path from --exec-path

2. gc execs 'git repack' (note: no dash).

3. Since there is a git in /foo (it's a build directory), /foo/git is

4. No explicit exec-path is set this time, hence, this secondary git sets

5. Since 'repack' is not a built-in, execv_dashed_external execs
   'git-repack' (note: dash).

6. There is a $pfxexecpath/git-repack, and it is taken.

7. This git-repack runs 'git pack-objects' (note: no dash).

8. There is no git in $pfxexecpath, but there is one in /foo. Hence,
   /foo/git is run.

9. pack-objects is a builtin, hence, in effect /foo/git-pack-objects
   is run.

As you can see, the way in which we previously set the PATH allowed to
mix gits of different vintage.  By setting GIT_EXEC_PATH when --exec-path
was given on the command line, we reduce the confusion.

Signed-off-by: Johannes Sixt <>
Signed-off-by: Junio C Hamano <>

index ce6741e..589059f 100644 (file)
@@ -25,6 +25,10 @@ void git_set_argv0_path(const char *path)
 void git_set_argv_exec_path(const char *exec_path)
        argv_exec_path = exec_path;
+       /*
+        * Propagate this setting to external programs.
+        */
+       setenv(EXEC_PATH_ENVIRONMENT, exec_path, 1);