From: Elijah Newren <newren@gmail.com>
Date: Tue, 30 Nov 2021 06:53:10 GMT
Subject: Re: [PATCH v4 01/11] t2501: add various tests for removing the current working directory
Message-ID: <CABPp-BHQsQqjCwjCUk-bSe_i52Wza-QgTDbHVsLn90D8q5FkuQ@mail.gmail.com>
In-Reply-To: <xmqqilwa2l8f.fsf@gitster.g>

On Mon, Nov 29, 2021 at 10:47 PM Junio C Hamano <gitster@pobox.com> wrote:
>
> "Elijah Newren via GitGitGadget" <gitgitgadget@gmail.com> writes:
>
> > +             # Although we want pwd & git status to pass, test for existing
> > +             # rather than desired behavior.
> > +             if test "$works" == "success"
>
> Don't double "=" here.  We are not writing for bash.  There are a
> few more instances of the same mistake in this patch.

Yep, also pointed out by Eric.  I've got a v5 queued up whose change
is just fixing this, but I'm waiting a bit to see if other items are
spotted in review before sending.

Thanks for spotting.