Skip to content

BLD: fix meson_version warning #28693

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Apr 12, 2025
Merged

Conversation

lucascolley
Copy link
Contributor

Closes gh-26223.

It looks like gh-26464 didn't deal with the warning in the linked issue because project.meson_version wasn't updated.

Version taken from https://github.com/numpy/meson/blob/0d93515fb826440d19707eee47fd92655fe2f166/mesonbuild/coredata.py#L75.

cc @charris

@github-actions github-actions bot added the 36 - Build Build related PR label Apr 12, 2025
@charris
Copy link
Member

charris commented Apr 12, 2025

Hmm, there will be a meson update in the near future, maybe we should wait on that. @rgommers Thoughts?

Copy link
Member

@rgommers rgommers left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is perfectly fine. It doesn't matter too much in practice, since this version check can't fail as long as we're using our vendored meson. But it's more accurate and does avoid the warning, so in it goes.

Thanks Lucas & Chuck.

@rgommers rgommers merged commit 3989a6e into numpy:main Apr 12, 2025
73 checks passed
@rgommers rgommers added this to the 2.3.0 release milestone Apr 12, 2025
@lucascolley lucascolley deleted the meson-version branch April 12, 2025 16:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
36 - Build Build related PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Should we update required meson version?
3 participants