From d382def1367e9d9b288e9b04b6f2da5987052b5b Mon Sep 17 00:00:00 2001 From: "Jason R. Coombs" Date: Wed, 19 Oct 2016 14:03:56 -0400 Subject: Use a meaningful variable name --- setuptools/dist.py | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/setuptools/dist.py b/setuptools/dist.py index dbb083bb..612040c8 100644 --- a/setuptools/dist.py +++ b/setuptools/dist.py @@ -105,15 +105,16 @@ def assert_string_list(dist, attr, value): def check_nsp(dist, attr, value): """Verify that namespace packages are valid""" - assert_string_list(dist, attr, value) - for nsp in value: + ns_packages = value + assert_string_list(dist, attr, ns_packages) + for nsp in ns_packages: if not dist.has_contents_for(nsp): raise DistutilsSetupError( "Distribution contains no modules or packages for " + "namespace package %r" % nsp ) parent, sep, child = nsp.rpartition('.') - if parent and parent not in value: + if parent and parent not in ns_packages: distutils.log.warn( "WARNING: %r is declared as a package namespace, but %r" " is not: please correct this in setup.py", nsp, parent -- cgit v1.2.1