From: Johannes Sixt <j6t@xxxxxxxx> When RUNTIME_PREFIX is enabled, the installation prefix is derived by trying a limited set of known locations where the git executable can reside. If none of these is found, a warning is emitted. When git is built in a directory that matches neither of these known names, the warning would always be emitted when the uninstalled executable is run. This is a problem on Windows, where gitk picks the uninstalled git when invoked from the build directory and gets confused by the warning. Print the warning only when GIT_TRACE is set. Signed-off-by: Johannes Sixt <j6t@xxxxxxxx> --- Johannes Schindelin schrieb: > P.S.: I cannot recall defending the warning, but I might have thought it a > good idea at some stage. This is no longer so. I do recall you defended it; good to know that you changed your mind. Thanks, -- Hannes exec_cmd.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/exec_cmd.c b/exec_cmd.c index 408e4e5..b2c07c7 100644 --- a/exec_cmd.c +++ b/exec_cmd.c @@ -28,7 +28,7 @@ const char *system_path(const char *path) !(prefix = strip_path_suffix(argv0_path, BINDIR)) && !(prefix = strip_path_suffix(argv0_path, "git"))) { prefix = PREFIX; - fprintf(stderr, "RUNTIME_PREFIX requested, " + trace_printf("RUNTIME_PREFIX requested, " "but prefix computation failed. " "Using static fallback '%s'.\n", prefix); } -- 1.7.0.83.g241b9.dirty -- 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