Re: [PATCH] upload-pack: allow stateless client EOF just prior to haves
- From
Eric Sunshine <sunshine@sunshineco.com>
- Date
- Oct 30, 2020, 02:59 UTC
- Message-ID
- <CAPig+cTOkd248WKh5KyVfScRf8fqcOQPj4V3waGQ3XWEcugiKg@mail.gmail.com>
- In-Reply-To
- <1604022059-18527-1-git-send-email-dan@mutual.io>
On Thu, Oct 29, 2020 at 9:51 PM Daniel Duvall <dan@mutual.io> wrote:
Show 16 quoted lines
> [...] > Instead, upload-pack should gently peek for an EOF between the sending > of shallow/unshallow lines (followed by flush) and the reading of client > haves. If the client has hung up at this point, exit normally. > > Signed-off-by: Daniel Duvall <dan@mutual.io> > --- > diff --git a/t/t9904-upload-pack-stateless-timely-eof.sh b/t/t9904-upload-pack-stateless-timely-eof.sh > @@ -0,0 +1,24 @@ > +#!/bin/sh > + > +test_description='stateless upload-pack gently handles EOF just after want/shallow/depth/flush' > + > +. ./test-lib.sh > + > +D=$(pwd)
What is the purpose of this assignment? It doesn't seem to be used anywhere in this script.
Show 16 quoted lines
> +test_expect_success 'upload-pack outputs flush and exits ok' ' > + test_commit initial && > + head=$(git rev-parse HEAD) && > + hexsz=$(test_oid hexsz) && > + > + printf "%04xwant %s\n%04xshallow %s\n000ddeepen 1\n0000" \ > + $(($hexsz + 10)) $head $(($hexsz + 13)) $head >request && > + > + git upload-pack --stateless-rpc "$(pwd)" <request >actual && > + > + printf "0000" >expect && > + > + test_cmp expect actual > +' > + > +test_done