On 6 Dec 2023, at 01:04, qi...@fujitsu.com via lists.openembedded.org 
<qiutt=fujitsu....@lists.openembedded.org> wrote:
> +-  prop = meson.get_external_property('ipc_rmid_deferred_release', 'auto')
> ++  prop = meson.get_external_property('ipc_rmid_deferred_release', 'false’)

Big NAK from me here.

This meson code _explicitly and intentionally_ doesn’t set a default.   If it 
can’t auto-detect then it’s on the builder to provide the right answer via a 
cross file.  I really dislike autoconf’s idiom of providing a default value and 
not making it clear unless you read the logs very carefully that a default was 
used which may be entirely incorrect: this behaviour is much better.

Running the test on my machine I discover that should be set to true, and this 
appears to be a Linux feature[1] instead of being specific to an architecture 
or libc.

So I recommend providing a cross file to Meson in target builds that sets this 
value to true.  systemd-boot, glib-2.0, and libsoup are all examples of recipes 
that already do this.

Ross

[1] 
https://gitlab.freedesktop.org/cairo/cairo/-/commit/5041b462d084de8552336275914d30c23bf5dd35
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#192081): 
https://lists.openembedded.org/g/openembedded-core/message/192081
Mute This Topic: https://lists.openembedded.org/mt/103004857/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to