Avoid misleading success message on error

When a push fails (for example when the remote head does not fast forward
to the desired ref) it is not correct to print "Everything up-to-date".

Signed-off-by: Johannes Schindelin <Johannes.Schindelin@gmx.de>
Signed-off-by: Junio C Hamano <junkio@cox.net>

diff --git a/send-pack.c b/send-pack.c
index a41bbe5ecfdff6fd7cea1bfc3d3415ba57867f2b..5bc2f017bca8e9c92352b19469b3c1021844ddec 100644 (file)
--- a/send-pack.c
+++ b/send-pack.c
@@ -272,7 +272,7 @@ static int send_pack(int in, int out, int nr_refspec, char **refspec)
        packet_flush(out);
        if (new_refs)
                pack_objects(out, remote_refs);
        packet_flush(out);
        if (new_refs)
                pack_objects(out, remote_refs);

-       else
+       else if (ret == 0)

                fprintf(stderr, "Everything up-to-date\n");
        close(out);
        return ret;
                fprintf(stderr, "Everything up-to-date\n");
        close(out);
        return ret;