FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Use meson-python instead of setuptools for wheels by oscarbenjamin · Pull Request #129 · flintlib/python-flint · GitHub

Repository navigation

Use meson-python instead of setuptools for wheels - #129

Merged
oscarbenjamin merged 52 commits into
flintlib:masterfrom
oscarbenjamin:pr_meson_python
Apr 25, 2024
Merged

oscarbenjamin merged 52 commits into
flintlib:masterfrom
oscarbenjamin:pr_meson_python

Conversation

Copy link
Copy Markdown
Collaborator

This is just for wheel building for now and not building the dependencies.

Currently this works except on Windows and also there is a bug with meson-python and python 3.13.

Copy link
Copy Markdown
Collaborator Author

there is a bug with meson-python and python 3.13.

That bug is described in mesonbuild/meson#12401. It will be resolved somehow before Python 3.13 is released so it is not a significant problem.

oscarbenjamin commented Mar 21, 2024 •
edited
Loading

Copy link
Copy Markdown
Collaborator Author

Okay, this is working on Windows now.

It is still not working with Flint < 3.1.0 for some reason...

Copy link
Copy Markdown
Collaborator Author

Since GitHub did not create the automatic back-reference this is where the rest of the discussion happened about these changes: mesonbuild/meson-python#602

Copy link
Copy Markdown
Collaborator Author

I've opened another discussion in meson-python about editable installs on OSX: mesonbuild/meson-python#604

Copy link
Copy Markdown
Collaborator Author

The main outstanding problem here is just that building with Flint < 3.1.0 fails when using pkgconfig. I think that this is due to a bug that was fixed by flintlib/flint#1647.

Copy link
Copy Markdown
Collaborator Author

I think that this is due to a bug that was fixed by flintlib/flint#1647.

I can confirm that fixing the Flint 3.0.1 flint.pc pkgconfig file fixes this problem. The question then is just whether python-flint should drop support for Flint < 3.0.1 or if it would be better to add something to the meson build config that can fix this.

oscarbenjamin reopened this Apr 10, 2024
oscarbenjamin reopened this Apr 21, 2024
oscarbenjamin merged commit a65c88c into flintlib:master Apr 25, 2024
oscarbenjamin deleted the pr_meson_python branch April 25, 2024 08:49
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

1 participant


Back | FazBrowse Home | New Git URL