refactor ref status logic for pushing
authorTay Ray Chuan <rctay89@gmail.com>
Fri, 8 Jan 2010 02:12:42 +0000 (10:12 +0800)
committerJunio C Hamano <gitster@pobox.com>
Sun, 10 Jan 2010 07:34:10 +0000 (23:34 -0800)
commit20e8b465a53e651cc3f50bd60f39d577ecdb7722
tree601e0e19befeac61d5ac60b35bb2a588f1b738f3
parent7b69079be964ca0a09a1a7895455ba3df379984e
refactor ref status logic for pushing

Move the logic that detects up-to-date and non-fast-forward refs to a
new function in remote.[ch], set_ref_status_for_push().

Make transport_push() invoke set_ref_status_for_push() before invoking
the push_refs() implementation. (As a side-effect, the push_refs()
implementation in transport-helper.c now knows of non-fast-forward
pushes.)

Removed logic for detecting up-to-date refs from the push_refs()
implementation in transport-helper.c, as transport_push() has already
done so for it.

Make cmd_send_pack() invoke set_ref_status_for_push() before invoking
send_pack(), as transport_push() can't do it for send_pack() here.

Mark the test on the return status of non-fast-forward push to fail.
Git now exits with success, as transport.c::transport_push() does not
check for refs with status REF_STATUS_REJECT_NONFASTFORWARD nor does it
indicate rejected pushes with its return value.

Mark the test for ref status to succeed. As mentioned earlier, refs
might be marked as non-fast-forwards, triggering the push status
printing mechanism in transport.c.

Signed-off-by: Tay Ray Chuan <rctay89@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
builtin-send-pack.c
remote.c
remote.h
t/t5541-http-push.sh
transport-helper.c
transport.c