Code

Modify setup_path() to only add git_exec_path() to PATH
authorSteffen Prohaska <prohaska@zib.de>
Sun, 18 Jan 2009 12:00:13 +0000 (13:00 +0100)
committerJunio C Hamano <gitster@pobox.com>
Mon, 26 Jan 2009 08:26:05 +0000 (00:26 -0800)
commit8e3462837b0ace04357503a3f58802cc2231df29
tree24d1f1dfaf5875734b8c8649457dbecb9297fcf5
parent2fb3f6db96492b680899f9e40f434eeb4c778a84
Modify setup_path() to only add git_exec_path() to PATH

Searching git programs only in the highest priority location is
sufficient.  It does not make sense that some of the required
programs are located at the highest priority location but other
programs are picked up from a lower priority exec-path.  If
exec-path is overridden a complete set of commands should be
provided, otherwise several different versions could get mixed,
which is likely to cause confusion.

If a user explicitly overrides the default location (by --exec-path
or GIT_EXEC_PATH), we now expect that all the required programs are
found there.  Instead of adding the directories "argv_exec_path",
"getenv(EXEC_PATH_ENVIRONMENT)", and "system_path(GIT_EXEC_PATH)"
to PATH, we now rely on git_exec_path(), which implements the same
order, but only returns the highest priority location to search for
executables.

Accessing only the location with highest priority is also required
for testing executables built with RUNTIME_PREFIX.  The call to
system_path() should be avoided if RUNTIME_PREFIX is set and the
executable is not installed at its final destination.  Because we
test before installing, we want to avoid calling system_path()
during tests.  The modifications in this commit avoid calling
system_path(GIT_EXEC_PATH) if a higher-priority location is
provided, which is the case when running the tests.

Signed-off-by: Steffen Prohaska <prohaska@zib.de>
Acked-by: Johannes Sixt <j6t@kdbg.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
exec_cmd.c