-
-
Notifications
You must be signed in to change notification settings - Fork 18.3k
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
BLD: Add meson build scripts #49114
BLD: Add meson build scripts #49114
Conversation
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.
Lgtm
Thanks, just some questions to follow up |
|
||
def write_version_info(path): | ||
if os.environ.get("MESON_DIST_ROOT"): | ||
# raise ValueError("dist root is", os.environ.get("MESON_DIST_ROOT")) |
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.
If you could remove in a follow up that would be great
import argparse | ||
import os | ||
|
||
import versioneer |
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.
So this won't be our vendored versioneer?
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.
Oops. I forgot to update this when I moved it into scripts, will fix it in my next PR.
Unfortunately, I don't think we are able to use non-vendored versioneer.
* BLD: Add script to generate pxi from pxi.in for meson * BLD: Add scripts to generate the project version
doc/source/whatsnew/vX.X.X.rst
file if fixing a bug or adding a new feature.In preparation for the rest of the meson build system support.