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

Revert "Work around mpi4py installation issues from setuptools vendor… #604

Merged
merged 1 commit into from
Feb 9, 2022

Conversation

matthiasdiener
Copy link
Member

@matthiasdiener matthiasdiener commented Feb 8, 2022

Revert "Work around mpi4py installation issues from setuptools vendoring distutils (#585)"

This reverts commit 6ae2890.

The underlying setuptools bug has been fixed:
pypa/setuptools#2987

See also illinois-ceesd/emirge#144

…ing distutils (#585)"

This reverts commit 6ae2890.

The underlying setuptools bug has been fixed:
pypa/setuptools#2987
@matthiasdiener matthiasdiener self-assigned this Feb 8, 2022
Copy link
Member

@MTCam MTCam left a comment

Choose a reason for hiding this comment

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

👍

Copy link
Contributor

@inducer inducer left a comment

Choose a reason for hiding this comment

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

I'm guessing this fix has also been released in setuptools and made it to conda? If so, LGTM!

@matthiasdiener
Copy link
Member Author

I'm guessing this fix has also been released in setuptools and made it to conda? If so, LGTM!

Yes, this is fixed since 60.2.0 (current version is 60.8.2).

@matthiasdiener matthiasdiener merged commit dc79645 into main Feb 9, 2022
@matthiasdiener matthiasdiener deleted the undo-setuptools branch February 9, 2022 05:07
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