-
Notifications
You must be signed in to change notification settings - Fork 193
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
Get rid of numpy.distutils #2149
Conversation
0ff15be
to
3be9524
Compare
docs/DEVGUIDE.rst
Outdated
validation suite is run through the command:: | ||
<http://pytest.org/latest/>`_ package to manage test cases. | ||
|
||
All requirementsa r elisted in ``pythran/tests/requirements.txt``. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All requirementsa r elisted in ``pythran/tests/requirements.txt``. | |
All requirements are listed in ``pythran/tests/requirements.txt``. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This worked fine for SciPy, and the changes LGTM.
This adds a dependency on setuptools and get rids of references to numpy.distutils and (part of) distutils. Fix #2148
3be9524
to
5b76f66
Compare
Thanks @hroncok, @musicinmybrain and @rgommers for extra-testing this patch. |
@serge-sans-paille would you please consider releasing a new version that contains these changes? |
Since numpy 1.26.0, numpy.distutils is being removed in python 3.12, let's try to have a solution that work for 3.12 and previous version without numpy.distutils.
Fix #2148