-
Notifications
You must be signed in to change notification settings - Fork 3
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
Build babelized projects with meson #90
Conversation
This is WIP, but the project files are in the right place.
It looks like meson doesn't allow readme in dynamic metadata. Be sure to install _version.py.
I'm starting with Fortran, then I'll generalize to the other supported languages. |
I got help on this from https://stackoverflow.com/a/30517735.
This is a temporary fix until I can handle multiple components in the `meson.build` file.
This is a *meson-python* behavior. See https://meson-python.readthedocs.io/en/stable/how-to-guides/editable-installs.html
… into mdpiper/build-with-meson
@mdpiper We're getting close, I think. Some things we still need to do,
I'm sure there are other things but I just wanted to get these down before I forget about them. |
@mcflugen I'll remove the Makefile. Should I also remove MANIFEST.in from the template? |
Yes, but we'll have to make sure that everything is included in the source distribution that needs to be (the only extra stuff is the data folder, I think). Maybe something along the lines of, [tool.setuptools.package-data]
package_name = [
"data/component_1/*",
"data/component_2/*",
] |
I'll leave the I will update the README, though, because if a user does an editable install with pip, they need to set |
@mcflugen I think this is ready to go. What do you think? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
This PR changes the build system for babelized projects from setuptools to meson.
This fixes #77 and fixes #87.