Skip to content

Commit c90667f

Browse files
committed
Install ninja in CI job
1 parent 8b6f3d5 commit c90667f

File tree

1 file changed

+8
-5
lines changed

1 file changed

+8
-5
lines changed

.github/workflows/buildwheel.yml

Lines changed: 8 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -149,11 +149,14 @@ jobs:
149149
python-version: '3.12'
150150
- run: sudo apt-get update
151151
- run: sudo apt-get install libflint-dev
152-
# The versions here should be kept in sync with pyproject.toml.
153-
# Not sure if ninja should be listed as a requirement in pyproject.toml
154-
# as with newer meson-python it does not seem to be needed but this CI
155-
# job would fail without it being installed explicitly here...
156-
- run: 'pip install cython==3.0 meson-python'
152+
# The versions of cython and meson-python here should be kept in sync
153+
# with those in pyproject.toml so that we test the stated minimum
154+
# versions.
155+
#
156+
# We don't need to specify ninja as a requirement in pyproject.toml
157+
# because without --no-build-isolation meson-python handles it
158+
# automatically in get_requirements_for_build_wheel().
159+
- run: 'pip install cython==3.0 meson-python==0.13 ninja==1.8.2'
157160
- run: pip install --no-build-isolation .
158161
- run: python -m flint.test --verbose
159162

0 commit comments

Comments
 (0)