Skip to content

DEPS: Bump meson-python, lower pin Cython #62086

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

Open
wants to merge 17 commits into
base: main
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from 3 commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 1 addition & 2 deletions .github/actions/build_pandas/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -23,12 +23,11 @@ runs:
shell: bash -el {0}

- name: Build Pandas
# wno-error reason: https://github.com/cython/cython/issues/6870
run: |
if [[ ${{ inputs.editable }} == "true" ]]; then
EDITABLE="-e"
else
EDITABLE=""
fi
pip install $EDITABLE . --no-build-isolation -v --no-deps --config-settings=setup-args="-Werror" --config-settings=setup-args="-Wno-error=unused-parameter"
pip install $EDITABLE . --no-build-isolation -v --no-deps -Ccompile-args="-Werror"
Copy link
Member

@WillAyd WillAyd Aug 13, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This actually should stay as setup-args not compile-args. Although it will work in either case on Unix platforms, -Werror is a setup argument in Meson as well. Passing it as a setup argument is portable across platforms, whereas this same command will fail with MSVC compilers

See also https://mesonbuild.com/Commands.html#setup

shell: bash -el {0}
2 changes: 1 addition & 1 deletion ci/deps/actions-311-minimum_versions.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ dependencies:
# build dependencies
- versioneer
- cython>=3.1,<4.0.0a0
- meson>=1.6,<2
- meson>=1.2.1,<2
- meson-python>=0.17.1,<1

# test dependencies
Expand Down
2 changes: 1 addition & 1 deletion ci/deps/actions-311.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ dependencies:
# build dependencies
- versioneer
- cython>=3.1,<4.0.0a0
- meson>=1.6,<2
- meson>=1.2.1,<2
- meson-python>=0.17.1,<1

# test dependencies
Expand Down
2 changes: 1 addition & 1 deletion ci/deps/actions-312.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ dependencies:
# build dependencies
- versioneer
- cython>=3.1,<4.0.0a0
- meson>=1.6,<2
- meson>=1.2.1,<2
- meson-python>=0.17.1,<1

# test dependencies
Expand Down
2 changes: 1 addition & 1 deletion ci/deps/actions-313-downstream_compat.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ dependencies:
# build dependencies
- versioneer
- cython>=3.1,<4.0.0a0
- meson>=1.6,<2
- meson>=1.2.1,<2
- meson-python>=0.17.1,<1

# test dependencies
Expand Down
2 changes: 1 addition & 1 deletion ci/deps/actions-313-numpydev.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ dependencies:

# build dependencies
- versioneer
- meson>=1.6,<2
- meson>=1.2.1,<2
- meson-python>=0.17.1,<1
- cython>=3.1,<4.0.0a0

Expand Down
2 changes: 1 addition & 1 deletion ci/deps/actions-313-pyarrownightly.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ dependencies:

# build dependencies
- versioneer
- meson>=1.6,<2
- meson>=1.2.1,<2
- cython>=3.1,<4.0.0a0
- meson-python>=0.17.1,<1

Expand Down
2 changes: 1 addition & 1 deletion ci/deps/actions-313.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ dependencies:
# build dependencies
- versioneer
- cython>=3.1,<4.0.0a0
- meson>=1.6,<2
- meson>=1.2.1,<2
- meson-python>=0.17.1,<1

# test dependencies
Expand Down
2 changes: 1 addition & 1 deletion environment.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ dependencies:
# build dependencies
- versioneer
- cython>=3.1,<4.0.0a0
- meson>=1.6,<2
- meson>=1.2.1,<2
- meson-python>=0.17.1,<1

# test dependencies
Expand Down
6 changes: 6 additions & 0 deletions meson.build
Original file line number Diff line number Diff line change
Expand Up @@ -44,6 +44,12 @@ else
meson.add_dist_script(py, versioneer, '-o', '_version_meson.py')
endif

cc = meson.get_compiler('c')
add_project_arguments(
# https://github.com/cython/cython/issues/6870
cc.get_supported_arguments(['-Wno-error=unused-parameter']),
language: 'c',
)
cy = meson.get_compiler('cython')
if cy.version().version_compare('>=3.1.0')
add_project_arguments('-Xfreethreading_compatible=true', language: 'cython')
Expand Down
2 changes: 1 addition & 1 deletion pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
# See https://github.com/scipy/scipy/pull/12940 for the AIX issue.
requires = [
"meson-python>=0.17.1,<1",
"meson>=1.6,<2",
"meson>=1.2.1,<2",
"wheel",
"Cython>=3.1,<4.0.0a0", # Note: sync with setup.py, environment.yml and asv.conf.json
# Force numpy higher than 2.0rc1, so that built wheels are compatible
Expand Down
2 changes: 1 addition & 1 deletion requirements-dev.txt
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
pip
versioneer[toml]
cython>=3.1,<4.0.0a0
meson[ninja]>=1.6,<2
meson[ninja]>=1.2.1,<2
meson-python>=0.17.1,<1
pytest>=7.3.2
pytest-cov
Expand Down
Loading