Skip to content

MAINT: Delete numpy.distutils.compat#15255

Merged
mattip merged 2 commits intonumpy:masterfrom
eric-wieser:remove-numpy.distutils.compat
Jan 6, 2020
Merged

MAINT: Delete numpy.distutils.compat#15255
mattip merged 2 commits intonumpy:masterfrom
eric-wieser:remove-numpy.distutils.compat

Conversation

@eric-wieser
Copy link
Member

@eric-wieser eric-wieser commented Jan 6, 2020

get_exception has no callers as of this commit's parent (gh-15248), and is hard to argue as public API given the only docstring in the file refers to "inside distutils".

This compat layer has been unnecessary since Python 2.5 (final release 8.5 years ago), which numpy has not supported since 1.7.0 (released 6 years ago).

@mattip
Copy link
Member

mattip commented Jan 6, 2020

touching distutils is tricky. There are downstream projects that may use it. Maybe a release note would ease the transition

@eric-wieser eric-wieser force-pushed the remove-numpy.distutils.compat branch from c9da1e0 to 27ce08b Compare January 6, 2020 11:39
@eric-wieser
Copy link
Member Author

Updated with a release note.

`get_exception` has no callers as of this commit's parent, and is hard to argue as public API given the only docstring in the file refers to "inside distutils".
@eric-wieser eric-wieser force-pushed the remove-numpy.distutils.compat branch from 27ce08b to 679c834 Compare January 6, 2020 12:18
@mattip mattip merged commit 3e85f92 into numpy:master Jan 6, 2020
@mattip
Copy link
Member

mattip commented Jan 6, 2020

Thanks @eric-wieser

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants