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

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

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

UNH CI testing is ready for this change.

On Wed, Oct 16, 2024 at 9:08 AM Patrick Robb <probb@iol.unh.edu> wrote:

> UNH will be updated to be running from the linux setup script by the end
> of the day today, and I'll send another response here.
>
> On Sun, Oct 13, 2024 at 9:26 PM zhoumin <zhoumin@loongson.cn> wrote:
>
>> 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> 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: 6107 bytes --]