diff options
author | Junio C Hamano <gitster@pobox.com> | 2023-06-23 11:21:16 -0700 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2023-06-23 11:21:16 -0700 |
commit | dcedba13b38978f56e5b703bfaff379a86d6e52f (patch) | |
tree | a23ef3854f56ff5e45ba534b8eddd2ecd8732e4e /run-command.c | |
parent | 5ee8fcdabc25ade1f8c5295575a1c20ab4f6f347 (diff) | |
parent | 6d224ac286d62bb3a10c3697f220b0c10d4b5f51 (diff) | |
download | git-dcedba13b38978f56e5b703bfaff379a86d6e52f.tar.gz |
Merge branch 'rs/run-command-exec-error-on-noent'
Simplify error message when run-command fails to start a command.
* rs/run-command-exec-error-on-noent:
run-command: report exec error even on ENOENT
t1800: loosen matching of error message for bad shebang
Diffstat (limited to 'run-command.c')
-rw-r--r-- | run-command.c | 14 |
1 files changed, 3 insertions, 11 deletions
diff --git a/run-command.c b/run-command.c index 60c9419866..758f8534da 100644 --- a/run-command.c +++ b/run-command.c @@ -307,7 +307,6 @@ enum child_errcode { CHILD_ERR_DUP2, CHILD_ERR_CLOSE, CHILD_ERR_SIGPROCMASK, - CHILD_ERR_ENOENT, CHILD_ERR_SILENT, CHILD_ERR_ERRNO }; @@ -390,9 +389,6 @@ static void child_err_spew(struct child_process *cmd, struct child_err *cerr) case CHILD_ERR_SIGPROCMASK: error_errno("sigprocmask failed restoring signals"); break; - case CHILD_ERR_ENOENT: - error_errno("cannot run %s", cmd->args.v[0]); - break; case CHILD_ERR_SILENT: break; case CHILD_ERR_ERRNO: @@ -846,13 +842,9 @@ fail_pipe: execve(argv.v[0], (char *const *) argv.v, (char *const *) childenv); - if (errno == ENOENT) { - if (cmd->silent_exec_failure) - child_die(CHILD_ERR_SILENT); - child_die(CHILD_ERR_ENOENT); - } else { - child_die(CHILD_ERR_ERRNO); - } + if (cmd->silent_exec_failure && errno == ENOENT) + child_die(CHILD_ERR_SILENT); + child_die(CHILD_ERR_ERRNO); } atfork_parent(&as); if (cmd->pid < 0) |