Commit d382def1 authored by Jason R. Coombs's avatar Jason R. Coombs

Use a meaningful variable name

parent b5299c31
...@@ -105,15 +105,16 @@ def assert_string_list(dist, attr, value): ...@@ -105,15 +105,16 @@ def assert_string_list(dist, attr, value):
def check_nsp(dist, attr, value): def check_nsp(dist, attr, value):
"""Verify that namespace packages are valid""" """Verify that namespace packages are valid"""
assert_string_list(dist, attr, value) ns_packages = value
for nsp in value: assert_string_list(dist, attr, ns_packages)
for nsp in ns_packages:
if not dist.has_contents_for(nsp): if not dist.has_contents_for(nsp):
raise DistutilsSetupError( raise DistutilsSetupError(
"Distribution contains no modules or packages for " + "Distribution contains no modules or packages for " +
"namespace package %r" % nsp "namespace package %r" % nsp
) )
parent, sep, child = nsp.rpartition('.') parent, sep, child = nsp.rpartition('.')
if parent and parent not in value: if parent and parent not in ns_packages:
distutils.log.warn( distutils.log.warn(
"WARNING: %r is declared as a package namespace, but %r" "WARNING: %r is declared as a package namespace, but %r"
" is not: please correct this in setup.py", nsp, parent " is not: please correct this in setup.py", nsp, parent
......
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