diff options
| author | Jason R. Coombs <jaraco@jaraco.com> | 2018-02-10 23:40:27 -0500 |
|---|---|---|
| committer | Jason R. Coombs <jaraco@jaraco.com> | 2018-02-10 23:40:27 -0500 |
| commit | ae9709b31450cdff1c5d00aefcb2ad0d11999f12 (patch) | |
| tree | c19e7291fae6a993f57098c1498a0261fdf39d65 | |
| parent | 7d34b2d75fa19fa689fd3033a28fc53562ee6a98 (diff) | |
| download | python-setuptools-git-ae9709b31450cdff1c5d00aefcb2ad0d11999f12.tar.gz | |
Use filter
| -rw-r--r-- | pkg_resources/__init__.py | 27 |
1 files changed, 13 insertions, 14 deletions
diff --git a/pkg_resources/__init__.py b/pkg_resources/__init__.py index 7635a901..c32de489 100644 --- a/pkg_resources/__init__.py +++ b/pkg_resources/__init__.py @@ -2693,20 +2693,19 @@ class Distribution(object): environment markers and filter out any dependencies not matching the markers. """ - for extra in list(dm): - if extra: - new_extra = extra - reqs = dm.pop(extra) - if ':' in extra: - new_extra, marker = extra.split(':', 1) - if invalid_marker(marker): - # XXX warn - reqs = [] - elif not evaluate_marker(marker): - reqs = [] - new_extra = safe_extra(new_extra) or None - - dm.setdefault(new_extra, []).extend(reqs) + for extra in list(filter(None, dm)): + new_extra = extra + reqs = dm.pop(extra) + if ':' in extra: + new_extra, marker = extra.split(':', 1) + if invalid_marker(marker): + # XXX warn + reqs = [] + elif not evaluate_marker(marker): + reqs = [] + new_extra = safe_extra(new_extra) or None + + dm.setdefault(new_extra, []).extend(reqs) return dm def _build_dep_map(self): |
