Skip to content

build: Update min version of meson to 0.48#511

Merged
igaw merged 1 commit intolinux-nvme:masterfrom
igaw:build-min-verison-fix
Oct 28, 2022
Merged

build: Update min version of meson to 0.48#511
igaw merged 1 commit intolinux-nvme:masterfrom
igaw:build-min-verison-fix

Conversation

@igaw
Copy link
Copy Markdown
Collaborator

@igaw igaw commented Oct 27, 2022

python3.extension_module() is depending on gnu_symbol_visibility. So arguably, this has been from day 0.

Version 0.47 was released on Jul 02, 2018 and 0.48 on Sep 22,
2018. Even Debian oldstable ships 0.49.

Signed-off-by: Daniel Wagner dwagner@suse.de

Fixes: #496

python3.extension_module() is depending on gnu_symbol_visibility. So
arguably, this has been from day 0.

Version 0.47 was released on Jul 02, 2018 and 0.48 on Sep 22,
2018. Even Debian oldstable ships 0.49.

Signed-off-by: Daniel Wagner <dwagner@suse.de>
@igaw igaw merged commit 1dd2e76 into linux-nvme:master Oct 28, 2022
@igaw igaw deleted the build-min-verison-fix branch October 28, 2022 09:24
@eli-schwartz
Copy link
Copy Markdown
Contributor

While it's perfectly fine to bump the minimum version of Meson, you are not actually setting this kwarg. Meson's python module does this internally, via mesonbuild/meson#10369

The warning is actually bogus, but it's emitted in the backend not in the interpreter, so we do not actually know whether it was set by the python module or by the user's meson.build. Fun! Also way too annoying to fix, it will just fix itself as part of the pending rewrite in mesonbuild/meson#10714.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

meson minimum versioning warning

3 participants