aboutsummaryrefslogtreecommitdiffstats
path: root/setuptools/__init__.py
diff options
context:
space:
mode:
authorTim Heap <tim@timheap.me>2016-09-22 13:44:01 +1000
committerTim Heap <tim@timheap.me>2016-10-05 04:26:04 +0200
commit2ecc8a4428dbc75bdfd96681283c7235b120f4bd (patch)
tree963b828fdbe11e50538395809f939064aa9d5d96 /setuptools/__init__.py
parentcf469bf239cc6c0d4ebb766c5e2b6c6098e2b28b (diff)
downloadexternal_python_setuptools-2ecc8a4428dbc75bdfd96681283c7235b120f4bd.tar.gz
external_python_setuptools-2ecc8a4428dbc75bdfd96681283c7235b120f4bd.tar.bz2
external_python_setuptools-2ecc8a4428dbc75bdfd96681283c7235b120f4bd.zip
Find nested packages with excluded parent
`find_packages(exclude=['pkg'])` should still find and include the `pkg.subpkg` package. Fixes #808
Diffstat (limited to 'setuptools/__init__.py')
-rw-r--r--setuptools/__init__.py16
1 files changed, 10 insertions, 6 deletions
diff --git a/setuptools/__init__.py b/setuptools/__init__.py
index 892626e6..baec3884 100644
--- a/setuptools/__init__.py
+++ b/setuptools/__init__.py
@@ -75,13 +75,17 @@ class PackageFinder(object):
rel_path = os.path.relpath(full_path, where)
package = rel_path.replace(os.path.sep, '.')
- # Check if the directory is a package and passes the filters
- if ('.' not in dir
- and include(package)
- and not exclude(package)
- and cls._looks_like_package(full_path)):
+ # Skip directory trees that are not valid packages
+ if ('.' in dir or not cls._looks_like_package(full_path)):
+ continue
+
+ # Should this package be included?
+ if include(package) and not exclude(package):
yield package
- dirs.append(dir)
+
+ # Keep searching subdirectories, as there may be more packages
+ # down there, even if the parent was excluded.
+ dirs.append(dir)
@staticmethod
def _looks_like_package(path):