Meson get_cross_property deprecated

Meson get_cross_property deprecated

[Morten Brørup]

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

Hi Bruce,

 

Meson version 1.3.2 emits a notice that meson.get_cross_property is deprecated: 

 

NOTICE: Future-deprecated features used:

* 0.58.0: {'meson.get_cross_property'}

 

It seems to have been replaced by meson.get_external_property:

https://mesonbuild.com/Reference-manual_builtin_meson.html#mesonget_cross_property

 

 

Looking forward to seeing you again next week. J

 

 

Med venlig hilsen / Kind regards,

-Morten Brørup

 


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

Re: Meson get_cross_property deprecated

[Bruce Richardson]

On Fri, Sep 20, 2024 at 10:22:59AM +0200, Morten Brørup wrote:
>    Hi Bruce,
> 
> 
>    Meson version 1.3.2 emits a notice that meson.get_cross_property is
>    deprecated:
> 
> 
>    NOTICE: Future-deprecated features used:
> 
>    * 0.58.0: {'meson.get_cross_property'}
> 
> 
>    It seems to have been replaced by meson.get_external_property:
> 
>    [1]https://mesonbuild.com/Reference-manual_builtin_meson.html#mesonget_
>    cross_property
> 
> 
Unfortunately, according to the docs, get_external_property is only
supported from 0.54 onwards, so we can't update to use it until we do a
meson minimum-version bump.

Cc: techboard
Question for community & techboard: is it better to bump minimum version
now in 24.11, or to wait till after the LTS release?  Thoughts?

/Bruce

RE: Meson get_cross_property deprecated

[Morten Brørup]

> From: Bruce Richardson [mailto:bruce.richardson@intel.com]
> Sent: Friday, 20 September 2024 11.08
> 
> On Fri, Sep 20, 2024 at 10:22:59AM +0200, Morten Brørup wrote:
> >    Hi Bruce,
> >
> >
> >    Meson version 1.3.2 emits a notice that meson.get_cross_property is
> >    deprecated:
> >
> >
> >    NOTICE: Future-deprecated features used:
> >
> >    * 0.58.0: {'meson.get_cross_property'}
> >
> >
> >    It seems to have been replaced by meson.get_external_property:
> >
> >    [1]https://mesonbuild.com/Reference-manual_builtin_meson.html#mesonget_
> >    cross_property
> >
> >
> Unfortunately, according to the docs, get_external_property is only
> supported from 0.54 onwards, so we can't update to use it until we do a
> meson minimum-version bump.
> 
> Cc: techboard
> Question for community & techboard: is it better to bump minimum version
> now in 24.11, or to wait till after the LTS release?  Thoughts?
> 
> /Bruce

I think we should bump the minimum Meson version now, or we will be stuck with it for the lifetime of the LTS release.

Also applies to minimum Python version, discussed in another thread:
https://inbox.dpdk.org/dev/ZuwfsT6iRqcjnxRe@bricha3-mobl1.ger.corp.intel.com/

-Morten

Re: Meson get_cross_property deprecated

[Tyler Retzlaff]

On Fri, Sep 20, 2024 at 11:20:12AM +0200, Morten Brørup wrote:
> > From: Bruce Richardson [mailto:bruce.richardson@intel.com]
> > Sent: Friday, 20 September 2024 11.08
> > 
> > On Fri, Sep 20, 2024 at 10:22:59AM +0200, Morten Brørup wrote:
> > >    Hi Bruce,
> > >
> > >
> > >    Meson version 1.3.2 emits a notice that meson.get_cross_property is
> > >    deprecated:
> > >
> > >
> > >    NOTICE: Future-deprecated features used:
> > >
> > >    * 0.58.0: {'meson.get_cross_property'}
> > >
> > >
> > >    It seems to have been replaced by meson.get_external_property:
> > >
> > >    [1]https://mesonbuild.com/Reference-manual_builtin_meson.html#mesonget_
> > >    cross_property
> > >
> > >
> > Unfortunately, according to the docs, get_external_property is only
> > supported from 0.54 onwards, so we can't update to use it until we do a
> > meson minimum-version bump.
> > 
> > Cc: techboard
> > Question for community & techboard: is it better to bump minimum version
> > now in 24.11, or to wait till after the LTS release?  Thoughts?

keep in mind windows build requires.. meson 0.57.0 due to bugs in newer
versions of meson.

> > 
> > /Bruce
> 
> I think we should bump the minimum Meson version now, or we will be stuck with it for the lifetime of the LTS release.
> 
> Also applies to minimum Python version, discussed in another thread:
> https://inbox.dpdk.org/dev/ZuwfsT6iRqcjnxRe@bricha3-mobl1.ger.corp.intel.com/
> 
> -Morten

Re: Meson get_cross_property deprecated

[Bruce Richardson]

On Mon, Sep 30, 2024 at 01:54:25PM -0700, Tyler Retzlaff wrote:
> On Fri, Sep 20, 2024 at 11:20:12AM +0200, Morten Brørup wrote:
> > > From: Bruce Richardson [mailto:bruce.richardson@intel.com]
> > > Sent: Friday, 20 September 2024 11.08
> > > 
> > > On Fri, Sep 20, 2024 at 10:22:59AM +0200, Morten Brørup wrote:
> > > >    Hi Bruce,
> > > >
> > > >
> > > >    Meson version 1.3.2 emits a notice that meson.get_cross_property is
> > > >    deprecated:
> > > >
> > > >
> > > >    NOTICE: Future-deprecated features used:
> > > >
> > > >    * 0.58.0: {'meson.get_cross_property'}
> > > >
> > > >
> > > >    It seems to have been replaced by meson.get_external_property:
> > > >
> > > >    [1]https://mesonbuild.com/Reference-manual_builtin_meson.html#mesonget_
> > > >    cross_property
> > > >
> > > >
> > > Unfortunately, according to the docs, get_external_property is only
> > > supported from 0.54 onwards, so we can't update to use it until we do a
> > > meson minimum-version bump.
> > > 
> > > Cc: techboard
> > > Question for community & techboard: is it better to bump minimum version
> > > now in 24.11, or to wait till after the LTS release?  Thoughts?
> 
> keep in mind windows build requires.. meson 0.57.0 due to bugs in newer
> versions of meson.
> 
Yep, thanks for the reminder of that. I've set that as the minimum version
in my proposed patchset [1]

/Bruce

[1] https://patches.dpdk.org/project/dpdk/patch/20240920125737.1197969-2-bruce.richardson@intel.com/