{"thread":{"id":"62835","subject":"[PATCH 1/2] Check for Perl 5.26.0 from Meson build","startedAt":"2025-01-20T16:04:28Z","lastAt":"2025-02-25T18:20:31Z","messageCount":16,"participants":["Peter Oliver","Patrick Steinhardt","Junio C Hamano"],"isPatch":true,"patchVersion":1,"patchTotal":2},"messages":[{"id":"510954","messageId":"20250120160301.121245-1-git@mavit.org.uk","threadId":"62835","inReplyTo":null,"subject":"[PATCH 1/2] Check for Perl 5.26.0 from Meson build","fromName":"Peter Oliver","fromEmail":"p.d.oliver@mavit.org.uk","sentAt":"2025-01-20T16:03:00Z","receivedAt":"2025-01-20T16:04:28Z","isPatch":true,"sender":{"key":"p.d.oliver@mavit.org.uk","avatar":null},"body":"Commit 702d8c1 (which landed not long before Meson was added) dropped\nsupport for Perl versions back to 5.8.1.\n\nSigned-off-by: Peter Oliver <git@mavit.org.uk>\n---\n meson.build | 2 +-\n 1 file changed, 1 insertion(+), 1 deletion(-)\n\ndiff --git a/meson.build b/meson.build\nindex 0064eb64f5..f01d81b39f 100644\n--- a/meson.build\n+++ b/meson.build\n@@ -755,7 +755,7 @@ endif\n \n # Note that we only set NO_PERL if the Perl features were disabled by the user.\n # It may not be set when we have found Perl, but only use it to run tests.\n-perl = find_program('perl', version: '>=5.8.1', dirs: program_path, required: perl_required)\n+perl = find_program('perl', version: '>=5.26.0', dirs: program_path, required: perl_required)\n perl_features_enabled = perl.found() and get_option('perl').allowed()\n if perl_features_enabled\n   build_options_config.set('NO_PERL', '')\n-- \n2.48.1\n\n"},{"id":"510955","messageId":"20250120160301.121245-2-git@mavit.org.uk","threadId":"62835","inReplyTo":"20250120160301.121245-1-git@mavit.org.uk","subject":"[PATCH 2/2] Fix Meson Perl version check","fromName":"Peter Oliver","fromEmail":"p.d.oliver@mavit.org.uk","sentAt":"2025-01-20T16:03:01Z","receivedAt":"2025-01-20T16:04:44Z","isPatch":true,"sender":{"key":"p.d.oliver@mavit.org.uk","avatar":null},"body":"Command `perl --version` says, e.g., “This is perl 5, version 26,\nsubversion 0 (v5.26.0)”, which Meson interprets as version 26.\n\nSigned-off-by: Peter Oliver <git@mavit.org.uk>\n---\n meson.build | 6 +++++-\n 1 file changed, 5 insertions(+), 1 deletion(-)\n\ndiff --git a/meson.build b/meson.build\nindex f01d81b39f..bf129205ad 100644\n--- a/meson.build\n+++ b/meson.build\n@@ -755,7 +755,11 @@ endif\n \n # Note that we only set NO_PERL if the Perl features were disabled by the user.\n # It may not be set when we have found Perl, but only use it to run tests.\n-perl = find_program('perl', version: '>=5.26.0', dirs: program_path, required: perl_required)\n+#\n+# When checking here, it would be better to say,\n+#   version: '>=5.26.0', version_argument: '-eprint \"$^V\"'\n+# but that requires Meson 1.5.0, which at the time of writing is rather new.\n+perl = find_program('perl', version: '>=26', dirs: program_path, required: perl_required)\n perl_features_enabled = perl.found() and get_option('perl').allowed()\n if perl_features_enabled\n   build_options_config.set('NO_PERL', '')\n-- \n2.48.1\n\n"},{"id":"510991","messageId":"Z49CQaW9n86Uml_l@pks.im","threadId":"62835","inReplyTo":"20250120160301.121245-1-git@mavit.org.uk","subject":"Re: [PATCH 1/2] Check for Perl 5.26.0 from Meson build","fromName":"Patrick Steinhardt","fromEmail":"ps@pks.im","sentAt":"2025-01-21T06:44:24Z","receivedAt":"2025-01-21T06:44:30Z","isPatch":true,"sender":{"key":"ps@pks.im","avatar":"https://avatars.githubusercontent.com/u/4056630?v=4"},"body":"On Mon, Jan 20, 2025 at 04:03:00PM +0000, Peter Oliver wrote:\n> Commit 702d8c1 (which landed not long before Meson was added) dropped\n> support for Perl versions back to 5.8.1.\n\nMakes sense. This commit was another thing that landed in parallel to\nthe Meson effort, so it's not surprising that Meson still requires the\nold version.\n\nA couple remarks for the commit message:\n\n  - The subject should typically start with the subsystem you're\n    touching, followed by a short description of what you're doing to\n    it.\n\n  - We aim to make the commit message work standalone, wihtout reading\n    the commit subject. So saying something along the lines that the\n    minimum required version in MEson hasn't been bumped accordingly\n    would make sense.\n\n  - We use `git log -1 --format=reference` when mentioning commits.\n\nSo, my suggestion:\n\n    meson: bump minimum required Perl version to 5.26.0\n\n    Commit 702d8c1f3b (Require Perl 5.26.0, 2024-10-23) dropped support\n    for Perl versions older than 5.28.0. The Meson build system, which\n    has been developed in parallel to that commit, hasn't been bumped\n    accordingly and thus still requires Perl 5.8.1 or newer.\n\n    Fix this and require Perl 5.28.0 or newer in Meson, as well.\n\nPatrick\n"},{"id":"510992","messageId":"Z49CTa-wconHakMs@pks.im","threadId":"62835","inReplyTo":"20250120160301.121245-2-git@mavit.org.uk","subject":"Re: [PATCH 2/2] Fix Meson Perl version check","fromName":"Patrick Steinhardt","fromEmail":"ps@pks.im","sentAt":"2025-01-21T06:44:29Z","receivedAt":"2025-01-21T06:44:33Z","isPatch":true,"sender":{"key":"ps@pks.im","avatar":"https://avatars.githubusercontent.com/u/4056630?v=4"},"body":"On Mon, Jan 20, 2025 at 04:03:01PM +0000, Peter Oliver wrote:\n> Command `perl --version` says, e.g., “This is perl 5, version 26,\n> subversion 0 (v5.26.0)”, which Meson interprets as version 26.\n\nYeah, I've also discovered this issue, but decided to improve it\nupstream in Meson [1]. Meson now prefers version numbers with dots when\nscanning through the output, which makes it match the \"v5.26.0\" part\ninstead of the \"version 26\" part. This will be released with Meson\n1.7.0, which should be released soonish.\n\nI'm a little torn whether it still makes sense to fix the issue in that\nlight. If we do, we should use a fix that properly fixes the underlying\nissue instead of adapting to the misdetected version, as that will stop\nworking once the mentioned fix is released.\n\nOne way would be to use `perl -V:version` instead of `perl --version`,\nwhich will print the following:\n\n    version='5.40.0';\n\nThe support of overriding the version argument has only been added\nrecently with Meson 1.5.0 though. We can still make it work though by\nmaking this conditional:\n\n    diff --git a/meson.build b/meson.build\n    index 07744c73b1..c452a79e37 100644\n    --- a/meson.build\n    +++ b/meson.build\n    @@ -754,7 +754,11 @@ endif\n     \n     # Note that we only set NO_PERL if the Perl features were disabled by the user.\n     # It may not be set when we have found Perl, but only use it to run tests.\n    -perl = find_program('perl', version: '>=5.8.1', dirs: program_path, required: perl_required)\n    +if meson.version().version_compare('>=1.5.0')\n    +  perl = find_program('perl', version: '>=5.8.1', dirs: program_path, required: perl_required, version_argument: '-V:version')\n    +else\n    +  perl = find_program('perl', version: '>=5.8.1', dirs: program_path, required: perl_required)\n    +endif\n     perl_features_enabled = perl.found() and get_option('perl').allowed()\n     if perl_features_enabled\n       build_options_config.set('NO_PERL', '')\n\nIs it worth it? I dunno. But also doesn't hurt if you want to go down\nthat path.\n\nThanks!\n\nPatrick\n\n[1]: https://github.com/mesonbuild/meson/commit/a3679a64eec7c312c81d657880f34f015426c7db\n"},{"id":"511174","messageId":"20250124163049.23965-1-git@mavit.org.uk","threadId":"62835","inReplyTo":"20250120160301.121245-1-git@mavit.org.uk","subject":"[PATCH v2 0/2] Fix Meson Perl version check","fromName":"Peter Oliver","fromEmail":"p.d.oliver@mavit.org.uk","sentAt":"2025-01-24T16:30:47Z","receivedAt":"2025-01-24T16:31:11Z","isPatch":true,"sender":{"key":"p.d.oliver@mavit.org.uk","avatar":null},"body":"As suggested, I have reworded the commit message of the first patch, and\nimproved the second patch to work correctly with future versions of\nMeson.\n\nPeter Oliver (2):\n  meson: bump minimum required Perl version to 5.26.0\n  meson: fix Perl version check for Meson versions before 1.7.0\n\n meson.build | 6 +++++-\n 1 file changed, 5 insertions(+), 1 deletion(-)\n\n-- \n2.48.1\n\n"},{"id":"511175","messageId":"20250124163049.23965-2-git@mavit.org.uk","threadId":"62835","inReplyTo":"20250124163049.23965-1-git@mavit.org.uk","subject":"[PATCH v2 1/2] meson: bump minimum required Perl version to 5.26.0","fromName":"Peter Oliver","fromEmail":"p.d.oliver@mavit.org.uk","sentAt":"2025-01-24T16:30:48Z","receivedAt":"2025-01-24T16:31:17Z","isPatch":true,"sender":{"key":"p.d.oliver@mavit.org.uk","avatar":null},"body":"Commit 702d8c1f3b (Require Perl 5.26.0, 2024-10-23) dropped support\nfor Perl versions older than 5.26.0. The Meson build system, which\nhas been developed in parallel to that commit, hasn't been bumped\naccordingly and thus still requires Perl 5.8.1 or newer.\n\nFix this by requiring Perl 5.26.0 or newer with Meson.\n\nSigned-off-by: Peter Oliver <git@mavit.org.uk>\n---\n meson.build | 2 +-\n 1 file changed, 1 insertion(+), 1 deletion(-)\n\ndiff --git a/meson.build b/meson.build\nindex 0064eb64f5..f01d81b39f 100644\n--- a/meson.build\n+++ b/meson.build\n@@ -755,7 +755,7 @@ endif\n \n # Note that we only set NO_PERL if the Perl features were disabled by the user.\n # It may not be set when we have found Perl, but only use it to run tests.\n-perl = find_program('perl', version: '>=5.8.1', dirs: program_path, required: perl_required)\n+perl = find_program('perl', version: '>=5.26.0', dirs: program_path, required: perl_required)\n perl_features_enabled = perl.found() and get_option('perl').allowed()\n if perl_features_enabled\n   build_options_config.set('NO_PERL', '')\n-- \n2.48.1\n\n"},{"id":"511176","messageId":"20250124163049.23965-3-git@mavit.org.uk","threadId":"62835","inReplyTo":"20250124163049.23965-1-git@mavit.org.uk","subject":"[PATCH v2 2/2] meson: fix Perl version check for Meson versions before 1.7.0","fromName":"Peter Oliver","fromEmail":"p.d.oliver@mavit.org.uk","sentAt":"2025-01-24T16:30:49Z","receivedAt":"2025-01-24T16:31:21Z","isPatch":true,"sender":{"key":"p.d.oliver@mavit.org.uk","avatar":null},"body":"Command `perl --version` says, e.g., “This is perl 5, version 26,\nsubversion 0 (v5.26.0)”, which older versions of Meson interpret as\nversion 26.\n\nThis will be fixed in Meson 1.7.0, but at the time of writing that isn’t\nyet released.\n\nIf we run `perl -V:version` we get the unambiguous response\n“version='5.26.0';”, but we need at least Meson 1.5.0 to be able to do that.\n\nSigned-off-by: Peter Oliver <git@mavit.org.uk>\n---\n meson.build | 6 +++++-\n 1 file changed, 5 insertions(+), 1 deletion(-)\n\ndiff --git a/meson.build b/meson.build\nindex f01d81b39f..80af578d36 100644\n--- a/meson.build\n+++ b/meson.build\n@@ -755,7 +755,11 @@ endif\n \n # Note that we only set NO_PERL if the Perl features were disabled by the user.\n # It may not be set when we have found Perl, but only use it to run tests.\n-perl = find_program('perl', version: '>=5.26.0', dirs: program_path, required: perl_required)\n+if meson.version().version_compare('>=1.5.0')\n+  perl = find_program('perl', dirs: program_path, required: perl_required, version: '>=5.26.0', version_argument: '-V:version')\n+else\n+  perl = find_program('perl', dirs: program_path, required: perl_required, version: '>=26')\n+endif\n perl_features_enabled = perl.found() and get_option('perl').allowed()\n if perl_features_enabled\n   build_options_config.set('NO_PERL', '')\n-- \n2.48.1\n\n"},{"id":"511227","messageId":"Z5c4OzzHWOo30Hu6@pks.im","threadId":"62835","inReplyTo":"20250124163049.23965-3-git@mavit.org.uk","subject":"Re: [PATCH v2 2/2] meson: fix Perl version check for Meson versions before 1.7.0","fromName":"Patrick Steinhardt","fromEmail":"ps@pks.im","sentAt":"2025-01-27T07:39:39Z","receivedAt":"2025-01-27T07:39:45Z","isPatch":true,"sender":{"key":"ps@pks.im","avatar":"https://avatars.githubusercontent.com/u/4056630?v=4"},"body":"On Fri, Jan 24, 2025 at 04:30:49PM +0000, Peter Oliver wrote:\n> diff --git a/meson.build b/meson.build\n> index f01d81b39f..80af578d36 100644\n> --- a/meson.build\n> +++ b/meson.build\n> @@ -755,7 +755,11 @@ endif\n>  \n>  # Note that we only set NO_PERL if the Perl features were disabled by the user.\n>  # It may not be set when we have found Perl, but only use it to run tests.\n> -perl = find_program('perl', version: '>=5.26.0', dirs: program_path, required: perl_required)\n> +if meson.version().version_compare('>=1.5.0')\n> +  perl = find_program('perl', dirs: program_path, required: perl_required, version: '>=5.26.0', version_argument: '-V:version')\n> +else\n> +  perl = find_program('perl', dirs: program_path, required: perl_required, version: '>=26')\n\nI'm still a bit sceptical whether we should adapt this second line to\nmatch against `>=26`. I guess as long as it doesn't break anything out\nthere it's okayish, and if we do see breakage we can in the worst case\nrevert it.\n\nIn any case, it deserves a comment why we're matching against the minor\nversion and not the whole version. E.g. something like this:\n\n    # Executing `perl --version` results in a string similar to the\n    # following output:\n    #\n    #     This is perl 5, version 40, subversion 0 (v5.40.0) built for x86_64-linux-thread-multi\n    #\n    # Meson picks up the \"40\" as version number instead of using \"v5.40.0\"\n    # due to the regular expression it uses. This got fixed in Meson 1.7.0,\n    # but meanwhile we have to either use `-V:version` instead of `--version`,\n    # which we can do starting with Meson 1.5.0 and newer, or we have to\n    # match against the minor version.\n\nThanks!\n\nPatrick\n"},{"id":"512603","messageId":"20250218153043.63535-1-git@mavit.org.uk","threadId":"62835","inReplyTo":"Z5c4OzzHWOo30Hu6@pks.im","subject":"[PATCH v3 0/2] Fix Meson Perl version check","fromName":"Peter Oliver","fromEmail":"p.d.oliver@mavit.org.uk","sentAt":"2025-02-18T15:30:41Z","receivedAt":"2025-02-18T15:30:58Z","isPatch":true,"sender":{"key":"p.d.oliver@mavit.org.uk","avatar":null},"body":"Suggested comments added.\n\nPeter Oliver (2):\n  meson: bump minimum required Perl version to 5.26.0\n  meson: fix Perl version check for Meson versions before 1.7.0\n\n meson.build | 17 ++++++++++++++++-\n 1 file changed, 16 insertions(+), 1 deletion(-)\n\n-- \n2.48.1\n\n"},{"id":"512604","messageId":"20250218153043.63535-2-git@mavit.org.uk","threadId":"62835","inReplyTo":"20250218153043.63535-1-git@mavit.org.uk","subject":"[PATCH v3 1/2] meson: bump minimum required Perl version to 5.26.0","fromName":"Peter Oliver","fromEmail":"p.d.oliver@mavit.org.uk","sentAt":"2025-02-18T15:30:42Z","receivedAt":"2025-02-18T15:31:04Z","isPatch":true,"sender":{"key":"p.d.oliver@mavit.org.uk","avatar":null},"body":"Commit 702d8c1f3b (Require Perl 5.26.0, 2024-10-23) dropped support\nfor Perl versions older than 5.26.0. The Meson build system, which\nhas been developed in parallel to that commit, hasn't been bumped\naccordingly and thus still requires Perl 5.8.1 or newer.\n\nFix this by requiring Perl 5.26.0 or newer with Meson.\n\nSigned-off-by: Peter Oliver <git@mavit.org.uk>\n---\n meson.build | 2 +-\n 1 file changed, 1 insertion(+), 1 deletion(-)\n\ndiff --git a/meson.build b/meson.build\nindex 0064eb64f5..f01d81b39f 100644\n--- a/meson.build\n+++ b/meson.build\n@@ -755,7 +755,7 @@ endif\n \n # Note that we only set NO_PERL if the Perl features were disabled by the user.\n # It may not be set when we have found Perl, but only use it to run tests.\n-perl = find_program('perl', version: '>=5.8.1', dirs: program_path, required: perl_required)\n+perl = find_program('perl', version: '>=5.26.0', dirs: program_path, required: perl_required)\n perl_features_enabled = perl.found() and get_option('perl').allowed()\n if perl_features_enabled\n   build_options_config.set('NO_PERL', '')\n-- \n2.48.1\n\n"},{"id":"512605","messageId":"20250218153043.63535-3-git@mavit.org.uk","threadId":"62835","inReplyTo":"20250218153043.63535-1-git@mavit.org.uk","subject":"[PATCH v3 2/2] meson: fix Perl version check for Meson versions before 1.7.0","fromName":"Peter Oliver","fromEmail":"p.d.oliver@mavit.org.uk","sentAt":"2025-02-18T15:30:43Z","receivedAt":"2025-02-18T15:31:05Z","isPatch":true,"sender":{"key":"p.d.oliver@mavit.org.uk","avatar":null},"body":"Command `perl --version` says, e.g., “This is perl 5, version 26,\nsubversion 0 (v5.26.0)”, which older versions of Meson interpret as\nversion 26.\n\nThis will be fixed in Meson 1.7.0, but at the time of writing that isn’t\nyet released.\n\nIf we run `perl -V:version` we get the unambiguous response\n“version='5.26.0';”, but we need at least Meson 1.5.0 to be able to do that.\n\nNote that Perl are seriously considering dropping the leading 5 entirely\nin the near future (https://perl.github.io/PPCs/ppc0025-perl-version/),\nbut that shouldn’t affect us.\n\nSigned-off-by: Peter Oliver <git@mavit.org.uk>\nCo-authored-by: Patrick Steinhardt <ps@pks.im>\n---\n meson.build | 17 ++++++++++++++++-\n 1 file changed, 16 insertions(+), 1 deletion(-)\n\ndiff --git a/meson.build b/meson.build\nindex f01d81b39f..fd19d348c6 100644\n--- a/meson.build\n+++ b/meson.build\n@@ -755,7 +755,22 @@ endif\n \n # Note that we only set NO_PERL if the Perl features were disabled by the user.\n # It may not be set when we have found Perl, but only use it to run tests.\n-perl = find_program('perl', version: '>=5.26.0', dirs: program_path, required: perl_required)\n+#\n+# At the time of writing, executing `perl --version` results in a string\n+# similar to the following output:\n+#\n+#     This is perl 5, version 40, subversion 0 (v5.40.0) built for x86_64-linux-thread-multi\n+#\n+# Meson picks up the \"40\" as version number instead of using \"v5.40.0\"\n+# due to the regular expression it uses. This got fixed in Meson 1.7.0,\n+# but meanwhile we have to either use `-V:version` instead of `--version`,\n+# which we can do starting with Meson 1.5.0 and newer, or we have to\n+# match against the minor version.\n+if meson.version().version_compare('>=1.5.0')\n+  perl = find_program('perl', dirs: program_path, required: perl_required, version: '>=5.26.0', version_argument: '-V:version')\n+else\n+  perl = find_program('perl', dirs: program_path, required: perl_required, version: '>=26')\n+endif\n perl_features_enabled = perl.found() and get_option('perl').allowed()\n if perl_features_enabled\n   build_options_config.set('NO_PERL', '')\n-- \n2.48.1\n\n"},{"id":"512635","messageId":"xmqq8qq3kqyk.fsf@gitster.g","threadId":"62835","inReplyTo":"20250218153043.63535-1-git@mavit.org.uk","subject":"Re: [PATCH v3 0/2] Fix Meson Perl version check","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2025-02-18T19:12:03Z","receivedAt":"2025-02-18T19:12:06Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Peter Oliver <p.d.oliver@mavit.org.uk> writes:\n\n> Suggested comments added.\n>\n> Peter Oliver (2):\n>   meson: bump minimum required Perl version to 5.26.0\n>   meson: fix Perl version check for Meson versions before 1.7.0\n>\n>  meson.build | 17 ++++++++++++++++-\n>  1 file changed, 16 insertions(+), 1 deletion(-)\n\nI wasn't involved in the review of the previous rounds (so it was a\nbit of surprise for me to be listed on the To: line), but it seems\nthat Patrick and you polished these two together, so I'll see if I\ncan add Patrick's acked-by/reviewed-by while queuing by waiting for\nthe earth to turn one rotation.\n\nThanks.\n"},{"id":"512655","messageId":"Z7WFpu6QEBJXwAIH@pks.im","threadId":"62835","inReplyTo":"xmqq8qq3kqyk.fsf@gitster.g","subject":"Re: [PATCH v3 0/2] Fix Meson Perl version check","fromName":"Patrick Steinhardt","fromEmail":"ps@pks.im","sentAt":"2025-02-19T07:17:58Z","receivedAt":"2025-02-19T07:18:07Z","isPatch":true,"sender":{"key":"ps@pks.im","avatar":"https://avatars.githubusercontent.com/u/4056630?v=4"},"body":"On Tue, Feb 18, 2025 at 11:12:03AM -0800, Junio C Hamano wrote:\n> Peter Oliver <p.d.oliver@mavit.org.uk> writes:\n> \n> > Suggested comments added.\n> >\n> > Peter Oliver (2):\n> >   meson: bump minimum required Perl version to 5.26.0\n> >   meson: fix Perl version check for Meson versions before 1.7.0\n> >\n> >  meson.build | 17 ++++++++++++++++-\n> >  1 file changed, 16 insertions(+), 1 deletion(-)\n> \n> I wasn't involved in the review of the previous rounds (so it was a\n> bit of surprise for me to be listed on the To: line), but it seems\n> that Patrick and you polished these two together, so I'll see if I\n> can add Patrick's acked-by/reviewed-by while queuing by waiting for\n> the earth to turn one rotation.\n\nYup, this version looks good to me, thanks Peter. So please feel free to\nadd either my Acked-by or Reviewed-by, I'd be fine with either.\n\nPatrick\n"},{"id":"512716","messageId":"xmqqo6yxhqdr.fsf@gitster.g","threadId":"62835","inReplyTo":"Z7WFpu6QEBJXwAIH@pks.im","subject":"Re: [PATCH v3 0/2] Fix Meson Perl version check","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2025-02-19T16:05:04Z","receivedAt":"2025-02-19T16:05:07Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Patrick Steinhardt <ps@pks.im> writes:\n\n> On Tue, Feb 18, 2025 at 11:12:03AM -0800, Junio C Hamano wrote:\n>> Peter Oliver <p.d.oliver@mavit.org.uk> writes:\n>> \n>> > Suggested comments added.\n>> >\n>> > Peter Oliver (2):\n>> >   meson: bump minimum required Perl version to 5.26.0\n>> >   meson: fix Perl version check for Meson versions before 1.7.0\n>> >\n>> >  meson.build | 17 ++++++++++++++++-\n>> >  1 file changed, 16 insertions(+), 1 deletion(-)\n>> \n>> I wasn't involved in the review of the previous rounds (so it was a\n>> bit of surprise for me to be listed on the To: line), but it seems\n>> that Patrick and you polished these two together, so I'll see if I\n>> can add Patrick's acked-by/reviewed-by while queuing by waiting for\n>> the earth to turn one rotation.\n>\n> Yup, this version looks good to me, thanks Peter. So please feel free to\n> add either my Acked-by or Reviewed-by, I'd be fine with either.\n\nThanks for working well together.  Let me mark the topic for 'next'.\n\n"},{"id":"513008","messageId":"ac54ce2d-a00d-4e9d-2c79-fa6b8de4aba4@mavit.org.uk","threadId":"62835","inReplyTo":"xmqq8qq3kqyk.fsf@gitster.g","subject":"Re: [PATCH v3 0/2] Fix Meson Perl version check","fromName":"Peter Oliver","fromEmail":"p.d.oliver@mavit.org.uk","sentAt":"2025-02-25T12:28:32Z","receivedAt":"2025-02-25T12:28:36Z","isPatch":true,"sender":{"key":"p.d.oliver@mavit.org.uk","avatar":null},"body":"On Tue, 18 Feb 2025, Junio C Hamano wrote:\n\n> I wasn't involved in the review of the previous rounds (so it was a\n> bit of surprise for me to be listed on the To: line)\n\nApologies if that wasn’t the correct process.  I am not a regular contributor, and was trying to follow the process described at https://git-scm.com/docs/SubmittingPatches#_choosing_your_reviewers, “After the list reached a consensus…”.\n\n-- \nPeter Oliver"},{"id":"513034","messageId":"xmqqmse9khsj.fsf@gitster.g","threadId":"62835","inReplyTo":"ac54ce2d-a00d-4e9d-2c79-fa6b8de4aba4@mavit.org.uk","subject":"Re: [PATCH v3 0/2] Fix Meson Perl version check","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2025-02-25T18:20:28Z","receivedAt":"2025-02-25T18:20:31Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Peter Oliver <p.d.oliver@mavit.org.uk> writes:\n\n> On Tue, 18 Feb 2025, Junio C Hamano wrote:\n>\n>> I wasn't involved in the review of the previous rounds (so it was a\n>> bit of surprise for me to be listed on the To: line)\n>\n> Apologies if that wasn’t the correct process.  I am not a regular\n> contributor, and was trying to follow the process described at\n> https://git-scm.com/docs/SubmittingPatches#_choosing_your_reviewers,\n> “After the list reached a consensus…”.\n\nNo apology needed.  Thanks for the update.\n\n"}]}