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

[zhoumin <zhoumin@loongson.cn>]

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

Hi Patrick Robb,

Thank you for your reminding. I have set the meson version to 0.57.0 in 
Loongson lab.

On Wed, Oct 9, 2024 at 2:31PM, Patrick Robb wrote:
> 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 
> <mailto: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 <mailto: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: 5416 bytes --]