Skip to content

Commit

Permalink
refine loongarch pin to apply only for py>=3.8
Browse files Browse the repository at this point in the history
Co-Authored-By: Randy Döring <[email protected]>
  • Loading branch information
h-vetinari and radoering committed Jun 30, 2023
1 parent adf7141 commit 305fe1b
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion setup.cfg
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ install_requires =
# loongarch64 requires numpy>=1.22.0
# Note that 1.22.0 broke support for using the Python limited C API
# (https://github.com/numpy/numpy/pull/20818), so we use 1.22.2 instead
numpy==1.22.2; platform_machine=='loongarch64' and python_version<'3.11'
numpy==1.22.2; platform_machine=='loongarch64' and python_version>='3.8' and python_version<'3.11'

# win-arm64; in the absence of information to the contrary, we want to use the default pins below;
# however, these are excluded due to `platform_machine not in 'arm64|...'`, so its easier to
Expand Down
2 changes: 1 addition & 1 deletion tests/test_dependencies.py
Original file line number Diff line number Diff line change
Expand Up @@ -90,7 +90,7 @@ def test_has_at_most_one_pinned_dependency(
case ["Linux", "s390x"]:
expect_pin = True # as of 1.17.5
case ["Linux", "loongarch64"]:
expect_pin = True # as of 1.22
expect_pin = (python_version != "3.7") # as of 1.22
# TODO: verify architectures for AIX/OS400
case ["AIX", ("x86" | "x86_64" | "s390x")]:
expect_pin = True # as of 1.16
Expand Down

0 comments on commit 305fe1b

Please sign in to comment.