BLD/DEV: switch to meson-python and pixi-build #583
Conversation
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
lucascolley
force-pushed
the
meson-python
branch
from
1e7212e to
b2ee65f
Compare
lucascolley
commented
Jan 5, 2026
lucascolley
force-pushed
the
meson-python
branch
4 times, most recently
from
a28bd6f to
db58474
Compare
This comment was marked as resolved.
This comment was marked as resolved.
lucascolley
commented
Jan 5, 2026
rgommers
reviewed
Jan 6, 2026
Member
rgommers
left a comment
rgommers
left a comment
There was a problem hiding this comment.
Thanks @lucascolley, this looks close. I didn't test yet, but hopefully the comments will be enough to resolve the remaining failures first.
lucascolley
force-pushed
the
meson-python
branch
from
b4efc62 to
bccf112
Compare
lucascolley
commented
Jan 6, 2026
Member
Author
lucascolley
left a comment
lucascolley
left a comment
There was a problem hiding this comment.
@rgommers ready to go I think!
| @@ -1,8 +1,8 @@ | |||
| 1. Update the version in `src/array_api_extra/__init__.py` | |||
| 1. Update the version in `pyproject.toml`, `meson.build`, and `src/array_api_extra/__init__.py` | |||
Member
Author
There was a problem hiding this comment.
up to 3 sources now, I don't think pixi-build is smart enough to handle dynamic versioning yet. But I think it's okay.
Comment on lines
43
to
54
| ### array-api-extra package definition ### | ||
|
|
||
| [tool.pixi.package.build.backend] | ||
| name = "pixi-build-python" | ||
| version = "*" | ||
|
|
||
| [tool.pixi.package.host-dependencies] | ||
| meson-python = "*" | ||
|
|
||
| [tool.pixi.package.run-dependencies] | ||
| array-api-compat = "*" |
Member
Author
There was a problem hiding this comment.
I switched over to pixi-build, much saner for ensuring we have ninja etc. in the environment when we need it.
Comment on lines
+55
to
+63
| ### workspace environments ### | ||
|
|
||
| [tool.pixi.environments] | ||
| default = { features = ["py313"], solve-group = "py313" } | ||
| lint = { features = ["py313", "lint"], solve-group = "py313" } | ||
| docs = { features = ["py313", "docs"], solve-group = "py313" } | ||
| tests = { features = ["py313", "tests"], solve-group = "py313" } | ||
| tests-py313 = { features = ["py313", "tests"], solve-group = "py313" } # alias of tests |
Member
Author
There was a problem hiding this comment.
this is just moved up from below, it is clearer to keep environment definitions at the top IMO
Comment on lines
+81
to
+87
| [tool.pixi.dev] | ||
| # this pulls in array-api-extra's host and run dependencies | ||
| array-api-extra.path = "." | ||
|
|
||
| [tool.pixi.dependencies] | ||
| array-api-compat = ">=1.12.0,<2" | ||
| array-api-extra.path = "." |
Member
Author
There was a problem hiding this comment.
ideally we wouldn't need both of these, I have asked offline
lucascolley
changed the title
lucascolley
force-pushed
the
meson-python
branch
from
bccf112 to
72c6a74
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
scipy/scipy#24271