summaryrefslogtreecommitdiff
path: root/builtin-receive-pack.c
diff options
context:
space:
mode:
authorJohannes Sixt <j6t@kdbg.org>2009-07-04 19:26:42 (GMT)
committerJunio C Hamano <gitster@pobox.com>2009-07-06 09:45:50 (GMT)
commitc024beb56da679839d61f352d088b9a86823233a (patch)
treece67d37c56575311d409d9b3090dd526d3178e9b /builtin-receive-pack.c
parentb99d5f40d6a5cba7d7cd7599063b3cd78aa4d219 (diff)
downloadgit-c024beb56da679839d61f352d088b9a86823233a.zip
git-c024beb56da679839d61f352d088b9a86823233a.tar.gz
git-c024beb56da679839d61f352d088b9a86823233a.tar.bz2
run_command: report failure to execute the program, but optionally don't
In the case where a program was not found, it was still the task of the caller to report an error to the user. Usually, this is an interesting case but only few callers actually reported a specific error (though many call sites report a generic error message regardless of the cause). With this change the error is reported by run_command, but since there is one call site in git.c that does not want that, an option is added to struct child_process, which is used to turn the error off. Signed-off-by: Johannes Sixt <j6t@kdbg.org> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin-receive-pack.c')
-rw-r--r--builtin-receive-pack.c4
1 files changed, 1 insertions, 3 deletions
diff --git a/builtin-receive-pack.c b/builtin-receive-pack.c
index 1dcdb1a..c85507b 100644
--- a/builtin-receive-pack.c
+++ b/builtin-receive-pack.c
@@ -125,9 +125,7 @@ static const char post_receive_hook[] = "hooks/post-receive";
static int run_status(int code, const char *cmd_name)
{
- if (code < 0 && errno == ENOENT)
- return error("execute of %s failed", cmd_name);
- else if (code > 0)
+ if (code > 0)
error("%s exited with error code %d", cmd_name, code);
return code;
}