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

Extension instance gets multiple manifestations when cython is present #488

Closed
ghost opened this issue Jan 24, 2016 · 5 comments
Closed

Comments

@ghost
Copy link

ghost commented Jan 24, 2016

Originally reported by: jaraco (Bitbucket: jaraco, GitHub: jaraco)


In this comment, @johnyf details an issue where the presence of Cython in the Python environment triggers a failure due to multiple manifestations of distutils.Extension or setuptools.Extension.


@ghost
Copy link
Author

ghost commented Jan 24, 2016

Original comment by jaraco (Bitbucket: jaraco, GitHub: jaraco):


I believe the issue here is related to how setuptools.sandbox removes itself from sys.modules in order to allow another version to be used (such as when upgrading setuptools).

I suspect that because setuptools imports cython in buildext, it should also remove it when hiding setuptools and for the same reasons.

@ghost
Copy link
Author

ghost commented Jan 24, 2016

Original comment by jaraco (Bitbucket: jaraco, GitHub: jaraco):


Also hide Cython when hiding setuptools, as setuptools will have imported Cython and Cython references the distutils.Extension. Fixes #488.

@ghost
Copy link
Author

ghost commented Jan 24, 2016

Original comment by jaraco (Bitbucket: jaraco, GitHub: jaraco):


@johnyf Would you please test with 19.6b1 and confirm it addresses the issue?

@ghost
Copy link
Author

ghost commented Jan 24, 2016

Original comment by johnyf (Bitbucket: johnyf, GitHub: johnyf):


As of 19.6b1 the issue appears to have been resolved, while it is present in 19.5.
Tested with Cython == 0.23.4, psutil == 3.4.2 and two packages:

  1. the minimal example described in 309
  2. dd with psutil in install_requires.

Thank you very much for the fix.

@ghost
Copy link
Author

ghost commented Jan 25, 2016

Original comment by jaraco (Bitbucket: jaraco, GitHub: jaraco):


Glad to hear it. 19.6 is released now.

@ghost ghost added major bug labels Mar 29, 2016
@ghost ghost closed this as completed Mar 29, 2016
jaraco added a commit that referenced this issue Apr 4, 2016
…rted Cython and Cython references the distutils.Extension. Fixes #488.
benoit-pierre added a commit to benoit-pierre/plover that referenced this issue May 21, 2016
Building pyobjc-framework-Cocoa fails if Cython is installed, see:

pypa/setuptools#488

This is fixed by upgrading setuptools to >=19.6.
benoit-pierre added a commit to benoit-pierre/plover that referenced this issue May 21, 2016
Building pyobjc-framework-Cocoa fails if Cython is installed, see:

pypa/setuptools#488

This is fixed by upgrading setuptools to >=19.6.
benoit-pierre added a commit to benoit-pierre/plover that referenced this issue May 21, 2016
Building pyobjc-framework-Cocoa fails if Cython is installed, see:

pypa/setuptools#488

This is fixed by upgrading setuptools to >=19.6.
TheaMorin pushed a commit to TheaMorin/plover that referenced this issue Mar 2, 2017
Building pyobjc-framework-Cocoa fails if Cython is installed, see:

pypa/setuptools#488

This is fixed by upgrading setuptools to >=19.6.
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

0 participants