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

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

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

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: 5697 bytes --]