enable "no-done" extension only when fetching over smart-http
authorJunio C Hamano <gitster@pobox.com>
Tue, 29 Mar 2011 17:16:29 +0000 (10:16 -0700)
committerJunio C Hamano <gitster@pobox.com>
Tue, 29 Mar 2011 19:19:58 +0000 (12:19 -0700)
commit8e9182e09127b20b8d8ce19655037991feac798d
treeb32ef068bd15b010aa9346ec10d2818618012d40
parent761ecf0bc7b6cddf311f00877c59e6381cdbdeea
enable "no-done" extension only when fetching over smart-http

When 'no-done' protocol extension is used, the upload-pack (i.e. the
server side) process stops listening to the fetch-pack after issuing the
final NAK, and starts sending the generated pack data back, but there may
be more "have" send by the latter in flight that the fetch-pack is
expecting to be responded with ACK/NAK.  This will typically result in a
deadlock (both will block on write that the other end never reads) or
SIGPIPE on the fetch-pack end (upload-pack will finish writing a small
pack and goes away).

Disable it unless fetch-pack is running under smart-http, where there is
no such streaming issue.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
Acked-by: Shawn O. Pearce <spearce@spearce.org>
builtin/fetch-pack.c