From: Andras Kucsma <r0maikx02b@xxxxxxxxx>
On Windows with git installed through cygwin, GIT_ASKPASS failed to run
for relative and absolute paths containing only backslashes as directory
separators.
The reason was that git assumed that if there are no forward slashes in
the executable path, it has to search for the executable on the PATH.
The fix is to look for OS specific directory separators, not just
forward slashes.
Signed-off-by: Andras Kucsma <r0maikx02b@xxxxxxxxx>
---
Fix dir sep handling of GIT_ASKPASS on Windows
On Windows with git installed through cygwin, GIT_ASKPASS failed to run
for relative and absolute paths containing only backslashes as directory
separators.
The reason was that git assumed that if there are no forward slashes in
the executable path, it has to search for the executable on the PATH.
The fix is to look for OS specific directory separators, not just
forward slashes.
Signed-off-by: Andras Kucsma r0maikx02b@xxxxxxxxx [r0maikx02b@xxxxxxxxx]
CC: Torsten Bögershausen tboegi@xxxxxx [tboegi@xxxxxx]
Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-587%2Fr0mai%2Ffix-prepare_cmd-windows-v1
Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-587/r0mai/fix-prepare_cmd-windows-v1
Pull-Request: https://github.com/gitgitgadget/git/pull/587
run-command.c | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/run-command.c b/run-command.c
index f5e1149f9b3..9fcc12ebf9c 100644
--- a/run-command.c
+++ b/run-command.c
@@ -421,12 +421,12 @@ static int prepare_cmd(struct argv_array *out, const struct child_process *cmd)
}
/*
- * If there are no '/' characters in the command then perform a path
- * lookup and use the resolved path as the command to exec. If there
- * are '/' characters, we have exec attempt to invoke the command
- * directly.
+ * If there are no dir separator characters in the command then perform
+ * a path lookup and use the resolved path as the command to exec. If
+ * there are dir separator characters, we have exec attempt to invoke
+ * the command directly.
*/
- if (!strchr(out->argv[1], '/')) {
+ if (find_last_dir_sep(out->argv[1]) == NULL) {
char *program = locate_in_PATH(out->argv[1]);
if (program) {
free((char *)out->argv[1]);
base-commit: 274b9cc25322d9ee79aa8e6d4e86f0ffe5ced925
--
gitgitgadget
