diff options
author | Tim Heap <tim@timheap.me> | 2016-09-22 13:44:01 +1000 |
---|---|---|
committer | Tim Heap <tim@timheap.me> | 2016-10-05 04:26:04 +0200 |
commit | 2ecc8a4428dbc75bdfd96681283c7235b120f4bd (patch) | |
tree | 963b828fdbe11e50538395809f939064aa9d5d96 /setuptools/__init__.py | |
parent | cf469bf239cc6c0d4ebb766c5e2b6c6098e2b28b (diff) | |
download | external_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__.py | 16 |
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): |