execvp will return EACCES when PATH entries are encountered that cannot be searched and the requested command isn't found. Since this error only arises in that particular case, warning the user is more appropriate than producing an error. Signed-off-by: Frans Klaver <fransklaver@xxxxxxxxx> --- run-command.c | 1 + t/t0061-run-command.sh | 3 ++- 2 files changed, 3 insertions(+), 1 deletions(-) diff --git a/run-command.c b/run-command.c index 17a65fe..ab14910 100644 --- a/run-command.c +++ b/run-command.c @@ -243,6 +243,7 @@ static void inspect_failure(const char *argv0, int silent_exec_failure) path = next + 1; if (!is_searchable(sb.buf)) { + warning("cannot search '%s'", sb.buf); strbuf_release(&sb); continue; } diff --git a/t/t0061-run-command.sh b/t/t0061-run-command.sh index 14b4ea6..a4585b0 100755 --- a/t/t0061-run-command.sh +++ b/t/t0061-run-command.sh @@ -62,7 +62,8 @@ test_expect_success POSIXPERM 'run_command reports EACCES, search path perms' ' test_must_fail test-run-command run-command hello.sh 2>err && chmod 755 inaccessible && - grep "fatal: cannot exec.*hello.sh" err + grep "fatal: cannot exec.*hello.sh" err && + grep "cannot search" err ' test_expect_success POSIXPERM 'run_command reports EACCES, interpreter fails' ' -- 1.7.8.1 -- 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