On Tue, Sep 17, 2024 at 1:46 PM karthik nayak <karthik.188@xxxxxxxxx> wrote:
> Patrick Steinhardt <ps@xxxxxx> writes:
> > +test_expect_success 'ref transaction: retry acquiring tables.list lock' '
> > + test_when_finished "rm -rf repo" &&
> > + git init repo &&
> > + (
> > + cd repo &&
> > + test_commit initial &&
> > + LOCK=.git/reftable/tables.list.lock &&
> > + >$LOCK &&
> > + {
> > + ( sleep 1 && rm -f $LOCK ) &
> > + } &&
> > + git -c reftable.lockTimeout=5000 update-ref refs/heads/branch HEAD
> > + )
> > +'
>
> Nit: This does stall the test for 1s. Which is slightly annoying when
> running single tests locally. Couldn't we achieve this by doing `sleep
> 0.1`?
`sleep 0.1` is neither POSIX nor portable.
