summaryrefslogtreecommitdiff
path: root/builtin-push.c
diff options
context:
space:
mode:
authorJeff King <peff@peff.net>2008-02-19 11:25:01 -0500
committerJunio C Hamano <gitster@pobox.com>2008-02-19 20:44:47 -0800
commit2b8130c338715936fcda82e734e76e86a33316aa (patch)
treeae312fd0c6daeccc0aa7b7d50da9d5717be56830 /builtin-push.c
parente5fc9a0aea2c3c49829b5cdf499339e5c759706b (diff)
downloadgit-2b8130c338715936fcda82e734e76e86a33316aa.tar.gz
push: indicate partialness of error message
The existing message indicates that an error occured during push, but it is unclear whether _any_ refs were actually pushed (even though the status table above shows which were pushed successfully and which were not, the message "failed to push" implies a total failure). By indicating that "some refs" failed, we hopefully indicate to the user that the table above contains the details. We could also put in an explicit "see above for details" message, but it seemed to clutter the output quite a bit (both on a line of its own, or at the end of the error line, which inevitably wraps). This could also be made more fancy if the transport mechanism passed back more details on how many refs succeeded and failed: error: failed to push %d out of %d refs to '%s' Signed-off-by: Jeff King <peff@peff.net> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin-push.c')
-rw-r--r--builtin-push.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/builtin-push.c b/builtin-push.c
index c8cb63e238..9f727c00f6 100644
--- a/builtin-push.c
+++ b/builtin-push.c
@@ -90,7 +90,7 @@ static int do_push(const char *repo, int flags)
if (!err)
continue;
- error("failed to push to '%s'", remote->url[i]);
+ error("failed to push some refs to '%s'", remote->url[i]);
errs++;
}
return !!errs;