Commit 57d450a4 authored by Jason R. Coombs's avatar Jason R. Coombs

Also hide Cython when hiding setuptools, as setuptools will have imported...

Also hide Cython when hiding setuptools, as setuptools will have imported Cython and Cython references the distutils.Extension. Fixes #488.
parent e7f9dab0
...@@ -16,6 +16,9 @@ CHANGES ...@@ -16,6 +16,9 @@ CHANGES
expose the desired functionality. For example:: expose the desired functionality. For example::
$ python -m setuptools.launch setup.py develop $ python -m setuptools.launch setup.py develop
* Issue #488: Fix dual manifestation of Extension class in
extension packages installed as dependencies when Cython
is present.
19.5 19.5
---- ----
......
...@@ -207,8 +207,12 @@ def _needs_hiding(mod_name): ...@@ -207,8 +207,12 @@ def _needs_hiding(mod_name):
True True
>>> _needs_hiding('distutils') >>> _needs_hiding('distutils')
True True
>>> _needs_hiding('os')
False
>>> _needs_hiding('Cython')
True
""" """
pattern = re.compile('(setuptools|pkg_resources|distutils)(\.|$)') pattern = re.compile('(setuptools|pkg_resources|distutils|Cython)(\.|$)')
return bool(pattern.match(mod_name)) return bool(pattern.match(mod_name))
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment