Patch 1 is a fix for what I described in [1]. The fix in patch 1 exposes a bug, which patch 2 fixes. Jonathan Nieder found the bug [2], and I have written a test covering this case. I've credited him as "Helped-by". [1] https://lore.kernel.org/git/20200423213735.242662-1-jonathantanmy@xxxxxxxxxx/ [2] https://lore.kernel.org/git/20200422165358.GB140314@xxxxxxxxxx/ Jonathan Tan (2): fetch-pack: in protocol v2, in_vain only after ACK fetch-pack: in protocol v2, reset in_vain upon ACK fetch-pack.c | 14 +++++++++---- t/t5500-fetch-pack.sh | 49 +++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 59 insertions(+), 4 deletions(-) -- 2.26.2.303.gf8c07b1a785-goog
