[Checkins] SVN: z3c.autoinclude/trunk/src/z3c/autoinclude/utils.py minor note: using setuptools within a library is Not Recommended. top_level is what i had been looking for, i think

Ethan Jucovy ejucovy at openplans.org
Mon Apr 21 12:07:33 EDT 2008


Log message for revision 85536:
  minor note: using setuptools within a library is Not Recommended. top_level is what i had been looking for, i think

Changed:
  U   z3c.autoinclude/trunk/src/z3c/autoinclude/utils.py

-=-
Modified: z3c.autoinclude/trunk/src/z3c/autoinclude/utils.py
===================================================================
--- z3c.autoinclude/trunk/src/z3c/autoinclude/utils.py	2008-04-21 16:02:03 UTC (rev 85535)
+++ z3c.autoinclude/trunk/src/z3c/autoinclude/utils.py	2008-04-21 16:07:32 UTC (rev 85536)
@@ -65,7 +65,8 @@
         for dist in dists:
             if not isUnzippedEgg(dist.location):
                 continue
-            packages = find_packages(dist.location)
+            packages = find_packages(dist.location) # TODO: don't use setuptools here; look for ``top_level.txt`` metadata instead
+            import pdb; pdb.set_trace()
             ns_packages = namespaceDottedNames(dist)
             if package_dottedname in ns_packages:
                 continue
@@ -73,7 +74,7 @@
                 continue
             valid_dists_for_package.append(dist)
     assert valid_dists_for_package, "No distributions found for package %s." % package_dottedname
-    assert len(valid_dists_for_package) == 1, "Multiple distributions found for package %s; z3c.autoinclude cowardly refuses to guess." % package_dottedname
+    assert len(valid_dists_for_package) == 1, "Multiple distributions found for package %s; z3c.autoinclude refuses to guess." % package_dottedname
     return valid_dists_for_package[0]
 
 def distributionForDottedName(dotted_name):



More information about the Checkins mailing list