Re: [PATCH v2] build: automatically report minimum meson version

[David Marchand <david.marchand@redhat.com>]

On Thu, May 15, 2025 at 11:37 AM Bruce Richardson
<bruce.richardson@intel.com> wrote:
>
> On Thu, May 15, 2025 at 09:09:22AM +0200, David Marchand wrote:
> > On Tue, May 13, 2025 at 11:00 AM Bruce Richardson
> > <bruce.richardson@intel.com> wrote:
> > >
> > > Add a script to buildtools to report the minimum meson version given in
> > > our meson.build file. Then use this script in two ways:
> > >
> > > 1. in the .ci/linux-setup.sh script, use the auto-determined minimum
> > >    version to set up the CI, rather than hard-coding it.
> > > 2. in meson.build call the script to report the version. This serves as
> > >    a sanity check to ensure that any changes to meson.build file do not
> > >    break the script.
> > >
> > > Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
> > > Acked-by: Stephen Hemminger <stephen@networkplumber.org>
> > > Reviewed-by: Patrick Robb <probb@iol.unh.edu>
> > > ---
> > > v2: rebased to latest main
> >
> > This patch is reverting dbcd72f3fba0 ("ci: fix unit tests with Fedora 37").
> >
> > There were some strange errors raised by UNH, though I don't see the
> > same python backtrace.
> > Let's try again:
> > Recheck-request: iol-unit-arm64-testing, iol-unit-amd64-testing
> >
> If we have issues using 0.57, I don't see there being issues with us
> marking the min meson version to 0.57.2 rather than 0.57 alone. I can do
> that in a v3, or else we can abandon this patch, if you prefer.

Retests show the same error, so I think we are hitting the issue fixed
with dbcd72f3fba0.

Bumping the minimal version to 0.57.2 seems fine.
I looked and can't find a distrib that ships meson 0.57.
So either a user relies on the distrib version (with a version >=
0.58, from my quick search) or the distrib provided version is too old
and then the user relies on pip.


-- 
David Marchand