Re: [PATCH 0/5] Increase minimum meson version

[Patrick Robb <probb@iol.unh.edu>]

[-- Attachment #1: Type: text/plain, Size: 3045 bytes --]

Hi Min Zhou,

I think it makes sense for you to set your meson version to the minimum
version supported for DPDK (so, .57 now I suppose) instead of .63. The
principle that David described above regarding LTS is also true for main.
So, for the main and next-* branch testing that Loongson lab does, we want
our CI testing to verify that no change relying on post .57 meson features
get merged into main.

On Tue, Oct 8, 2024 at 9:07 PM zhoumin <zhoumin@loongson.cn> wrote:

> Hi David,
>
> On Tues, Oct 8, 2024 at 8:28AM, David Marchand wrote:
> > Hello CI guys,
> >
> > On Fri, Sep 20, 2024 at 2:57 PM Bruce Richardson
> > <bruce.richardson@intel.com> wrote:
> >> This patchset proposed increasing the minimum meson version to 0.57
> >> and makes changes to update our build files appropriately for that
> >> change: replacing deprecated functions, removing unnecessary version
> >> checks and taking advantage of some new capabilities.
> >>
> >> Why 0.57? No one particular reason; it's mainly a conservative version
> >> bump that doesn't have many impacts, but still gives us the minimum
> >> updates we need to replace the deprecated get_cross_properties fn
> >> and have a few extra features guaranteed available.
> >>
> >> Bruce Richardson (5):
> >>    build: increase minimum meson version to 0.57
> >>    build: remove version check on compiler links function
> >>    build: remove unnecessary version checks
> >>    build: use version file support from meson
> >>    build: replace deprecated meson function
> >>
> >>   .ci/linux-setup.sh                        | 2 +-
> >>   config/arm/meson.build                    | 4 ++--
> >>   config/meson.build                        | 8 ++++----
> >>   config/riscv/meson.build                  | 4 ++--
> >>   doc/api/meson.build                       | 2 +-
> >>   doc/guides/linux_gsg/sys_reqs.rst         | 2 +-
> >>   doc/guides/prog_guide/build-sdk-meson.rst | 2 +-
> >>   drivers/common/qat/meson.build            | 2 +-
> >>   drivers/crypto/ipsec_mb/meson.build       | 2 +-
> >>   drivers/event/cnxk/meson.build            | 2 +-
> >>   drivers/meson.build                       | 7 ++-----
> >>   drivers/net/cnxk/meson.build              | 2 +-
> >>   lib/meson.build                           | 6 ------
> >>   meson.build                               | 7 ++-----
> >>   14 files changed, 20 insertions(+), 32 deletions(-)
> > This series can't be merged until the (UNH and LoongArch) CI are ready
> > for such a change.
> >
> > TL;DR: the meson minimum version is being changed from 0.53.2 to 0.57
> > in the current release.
> >
> > @UNH @Min Zhou
> > How long would it take for all CI to be ready for this change?
> It's OK for Loongson lab. The meson version for DPDK CI in Loongson lab
> is 0.63.0.
> > Important note: if relevant to your CI, testing against LTS branches
> > must still be done with the 0.53.2 version, so no change relying on
> > post 0.53.2 meson feature gets backported.
> >
> >
>
>

[-- Attachment #2: Type: text/html, Size: 3943 bytes --]