On Tue, May 26, 2015 at 3:19 PM, Junio C Hamano <gitster@xxxxxxxxx> wrote:
> Stefan Beller <sbeller@xxxxxxxxxx> writes:
>
>> @@ -175,7 +179,18 @@ int cmd_fetch_pack(int argc, const char **argv, const char *prefix)
>> if (!conn)
>> return args.diag_url ? 0 : 1;
>> }
>> - get_remote_heads(fd[0], NULL, 0, &ref, 0, NULL, &shallow);
>> +
>> + switch (args.version) {
>> + default:
>> + case 2:
>> + get_remote_capabilities(fd[0], NULL, 0);
>> + request_capabilities(fd[1]);
>> + break;
Actually this is wrong, we need to actually fall through from here as well,
so we not only talk capabilities negotiation, but then continue
with get_remote_heads.
>> + case 1: /* fall through */
>> + case 0:
>> + get_remote_heads(fd[0], NULL, 0, &ref, 0, NULL, &shallow);
>> + break;
>> + }
>
> Nice ;-)
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html