mbox series

[v3,0/2] Fix Meson Perl version check

Message ID 20250218153043.63535-1-git@mavit.org.uk (mailing list archive)
Headers show
Series Fix Meson Perl version check | expand

Message

Peter Oliver Feb. 18, 2025, 3:30 p.m. UTC
Suggested comments added.

Peter Oliver (2):
  meson: bump minimum required Perl version to 5.26.0
  meson: fix Perl version check for Meson versions before 1.7.0

 meson.build | 17 ++++++++++++++++-
 1 file changed, 16 insertions(+), 1 deletion(-)

Comments

Junio C Hamano Feb. 18, 2025, 7:12 p.m. UTC | #1
Peter Oliver <p.d.oliver@mavit.org.uk> writes:

> Suggested comments added.
>
> Peter Oliver (2):
>   meson: bump minimum required Perl version to 5.26.0
>   meson: fix Perl version check for Meson versions before 1.7.0
>
>  meson.build | 17 ++++++++++++++++-
>  1 file changed, 16 insertions(+), 1 deletion(-)

I wasn't involved in the review of the previous rounds (so it was a
bit of surprise for me to be listed on the To: line), but it seems
that Patrick and you polished these two together, so I'll see if I
can add Patrick's acked-by/reviewed-by while queuing by waiting for
the earth to turn one rotation.

Thanks.
Patrick Steinhardt Feb. 19, 2025, 7:17 a.m. UTC | #2
On Tue, Feb 18, 2025 at 11:12:03AM -0800, Junio C Hamano wrote:
> Peter Oliver <p.d.oliver@mavit.org.uk> writes:
> 
> > Suggested comments added.
> >
> > Peter Oliver (2):
> >   meson: bump minimum required Perl version to 5.26.0
> >   meson: fix Perl version check for Meson versions before 1.7.0
> >
> >  meson.build | 17 ++++++++++++++++-
> >  1 file changed, 16 insertions(+), 1 deletion(-)
> 
> I wasn't involved in the review of the previous rounds (so it was a
> bit of surprise for me to be listed on the To: line), but it seems
> that Patrick and you polished these two together, so I'll see if I
> can add Patrick's acked-by/reviewed-by while queuing by waiting for
> the earth to turn one rotation.

Yup, this version looks good to me, thanks Peter. So please feel free to
add either my Acked-by or Reviewed-by, I'd be fine with either.

Patrick
Junio C Hamano Feb. 19, 2025, 4:05 p.m. UTC | #3
Patrick Steinhardt <ps@pks.im> writes:

> On Tue, Feb 18, 2025 at 11:12:03AM -0800, Junio C Hamano wrote:
>> Peter Oliver <p.d.oliver@mavit.org.uk> writes:
>> 
>> > Suggested comments added.
>> >
>> > Peter Oliver (2):
>> >   meson: bump minimum required Perl version to 5.26.0
>> >   meson: fix Perl version check for Meson versions before 1.7.0
>> >
>> >  meson.build | 17 ++++++++++++++++-
>> >  1 file changed, 16 insertions(+), 1 deletion(-)
>> 
>> I wasn't involved in the review of the previous rounds (so it was a
>> bit of surprise for me to be listed on the To: line), but it seems
>> that Patrick and you polished these two together, so I'll see if I
>> can add Patrick's acked-by/reviewed-by while queuing by waiting for
>> the earth to turn one rotation.
>
> Yup, this version looks good to me, thanks Peter. So please feel free to
> add either my Acked-by or Reviewed-by, I'd be fine with either.

Thanks for working well together.  Let me mark the topic for 'next'.