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

Unconditionally require numpy #64

Closed
wants to merge 1 commit into from

Conversation

pabs3
Copy link
Contributor

@pabs3 pabs3 commented Feb 6, 2023

Not requiring it when it is already installed breaks Debian package builds,
which use setup.py install and translate requires to Debian package Depends.

SciPy has reverted back to unconditionally requiring numpy too.

CC: @TimothyFitz
Fixes: https://bugs.debian.org/1030653
See-also: scipy/scipy#12727
See-also: scipy/scipy#12729
See-also: scipy commit 988d69724c8ee963e45f8fedf9cba1ca18bf941d
Reverts: #38
Reverts: commit b0dde3b

Not requiring it when it is already installed breaks Debian package builds,
which use setup.py install and translate requires to Debian package Depends.

SciPy has reverted back to unconditionally requiring numpy too.

Fixes: https://bugs.debian.org/1030653
See-also: scipy/scipy#12727
See-also: scipy/scipy#12729
See-also: scipy commit 988d69724c8ee963e45f8fedf9cba1ca18bf941d
Reverts: wmayner#38
Reverts: commit b0dde3b.
@wmayner
Copy link
Owner

wmayner commented Feb 27, 2023

Thanks for the heads up on this.numpy is unconditionally required in the latest release.

@wmayner wmayner closed this Feb 27, 2023
@pabs3 pabs3 deleted the always-require-numpy branch February 27, 2023 05:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants