Re: [PATCH v2 0/2] Add new "signature" atom

From: Kousik Sanagavarapu <five231003@gmail.com>
Date: Jun 2, 2023, 13:13 UTC
Message-ID: <ZHnq-e0YAJSlAi9k@five231003>
In-Reply-To: <xmqqzg5i8gpt.fsf@gitster.g>

On Fri, Jun 02, 2023 at 04:29:34PM +0900, Junio C Hamano wrote:

Show 6 quoted lines

> Kousik Sanagavarapu <five231003@gmail.com> writes:
> 
> > I have also rebased this to be on top of v2.41.0, the previous version
> > was on top of v2.41.0-rc0.
> 
> I am still feverish and feeling weak so no real review from me yet,

Please take care and get well soon.

Show 14 quoted lines

> but there is one thing that immediately jumped at me.
>
> > Range-diff against v1:
> >
> > 1:  5c97d11b79 ! 1:  87465ef1a8 t/lib-gpg: introduce new prereq GPG2
> >     @@ t/lib-gpg.sh: test_lazy_prereq GPG '
> >      +  test $? != 127 || exit 1
> >      +
> >      +  case "$gpg_version" in
> >     -+  !"gpg (GnuPG) 2."*)
> >     ++  "gpg (GnuPG) 0."* | "gpg (GnuPG) 1.*")
> 
> The last '*' being inside double-quote would not be what you
> intended, I suspect?

Yeah, that was kind of a typo, thanks for catching it.

Thanks