Skip to content

Remove msvc hack for Numpy < 1.11.2#3741

Merged
jaraco merged 1 commit into
pypa:mainfrom
mgorny:msvc-hack
Jan 21, 2023
Merged

Remove msvc hack for Numpy < 1.11.2#3741
jaraco merged 1 commit into
pypa:mainfrom
mgorny:msvc-hack

Conversation

@mgorny

@mgorny mgorny commented Dec 30, 2022

Copy link
Copy Markdown
Contributor

Summary of changes

Remove the msvc hack that was applied if numpy.distutils were loaded prior to Numpy 1.11.2. These versions date back to June 2016 and no longer build with Python 3.7. At the same time, the hack relies on packaging.version.LegacyVersion that is removed in packaging 22.0.

Pull Request Checklist

@mgorny

mgorny commented Dec 30, 2022

Copy link
Copy Markdown
Contributor Author

Hmm, is CI broken?

@abravalheri

abravalheri commented Jan 7, 2023

Copy link
Copy Markdown
Contributor

Let's try now with a rebase.

@abravalheri

Copy link
Copy Markdown
Contributor

@Mergifyio rebase

@jaraco

jaraco commented Jan 16, 2023

Copy link
Copy Markdown
Member

Looking at the error message, I think this change introduces a fault. monkey.patch_all is no longer able to patch distutils with the removed function. Probably if the hack is no longer needed, it should also be removed from the monkeypatching of distutils.

@mgorny

mgorny commented Jan 21, 2023

Copy link
Copy Markdown
Contributor Author

Looking at the error message, I think this change introduces a fault. monkey.patch_all is no longer able to patch distutils with the removed function. Probably if the hack is no longer needed, it should also be removed from the monkeypatching of distutils.

Thanks. My grep-foo has been poor there.

Remove the msvc hack that was applied if numpy.distutils were loaded
prior to Numpy 1.11.2.  These versions date back to June 2016 and no
longer build with Python 3.7.  At the same time, the hack relies
on packaging.version.LegacyVersion that is removed in packaging 22.0.
@mgorny

mgorny commented Jan 21, 2023

Copy link
Copy Markdown
Contributor Author

Ok, I think it's good now.

@jaraco jaraco merged commit 40911c1 into pypa:main Jan 21, 2023
@mgorny mgorny deleted the msvc-hack branch January 21, 2023 16:25
@mgorny

mgorny commented Jan 21, 2023

Copy link
Copy Markdown
Contributor Author

Thanks!

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.

3 participants