Skip to content
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

build: use oldest-supported-numpy #165

Merged
merged 1 commit into from
Apr 19, 2021
Merged

build: use oldest-supported-numpy #165

merged 1 commit into from
Apr 19, 2021

Conversation

MuellerSeb
Copy link
Member

@MuellerSeb MuellerSeb commented Apr 17, 2021

Closes #164

Use the meta package for numpy as a build dep: https://github.com/scipy/oldest-supported-numpy

See:
https://scikit-hep.org/developer/packaging#special-additions-numpy

The reason to use the oldest available Numpy version as a build-time dependency is because of ABI compatibility. Binaries compiled with old Numpy versions are binary compatible with newer Numpy versions, but not vice versa.

@MuellerSeb MuellerSeb added bug Something isn't working Installation Problems during the installation of GSTools. CI/CD labels Apr 17, 2021
@MuellerSeb MuellerSeb requested a review from LSchueler April 17, 2021 23:45
@MuellerSeb MuellerSeb self-assigned this Apr 17, 2021
@MuellerSeb MuellerSeb added this to the 1.3.1 milestone Apr 18, 2021
@MuellerSeb MuellerSeb mentioned this pull request Apr 18, 2021
Copy link
Member

@LSchueler LSchueler left a comment

Choose a reason for hiding this comment

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

That's a nice and easy fix, thanks for handling this!

@MuellerSeb MuellerSeb merged commit 23ff228 into develop Apr 19, 2021
@MuellerSeb MuellerSeb deleted the numpy_build_fix branch April 19, 2021 09:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working CI/CD Installation Problems during the installation of GSTools.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

DLL load failed
2 participants