@matheusbsbs / matheus-bagliano / commits / 11c7f2a30b
Fork 0

Revert "fetch: default to protocol version 2"

This reverts commit 684ceae32dae726c6a5c693b257b156926aba8b7. Users fetching from linux-next and other kernel remotes are reporting that the limited ref advertisement causes negotiation to reach MAX_IN_VAIN, resulting in too-large fetches. Reported-by: Lubomir Rintel <lkundrak@v3.sk> Reported-by: "Dixit, Ashutosh" <ashutosh.dixit@intel.com> Reported-by: Jiri Slaby <jslaby@suse.cz> Reported-by: Konstantin Ryabitsev <konstantin@linuxfoundation.org> Signed-off-by: Jonathan Nieder <jrnieder@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>

Jonathan Nieder committed Apr 22, 2020 at 08:50 UTC 11c7f2a30b9dadcccc7bde66a34e0cb0cb5cf52c
2 files changed +2 -2
Documentation/config/protocol.txt
+1 -1
@@ -48,7 +48,7 @@ protocol.version::
48 If set, clients will attempt to communicate with a server
49 using the specified protocol version. If the server does
50 not support it, communication falls back to version 0.
51 - If unset, the default is `2`.
51 + If unset, the default is `0`.
52 Supported versions:
53 +
54 --
protocol.c
+1 -1
@@ -39,7 +39,7 @@ enum protocol_version get_protocol_version_config(void)
39 return env;
40 }
41
42 - return protocol_v2;
42 + return protocol_v0;
43 }
44
45 enum protocol_version determine_protocol_version_server(void)