Browse files

Pass the ignore argument to the recursive call, otherwise we can only…

… ignore top level packages
  • Loading branch information...
1 parent 892d071 commit 5611bfaf64253939ceb696de5137a9ceb2b84568 @lorenzogil lorenzogil committed Sep 23, 2012
Showing with 1 addition and 1 deletion.
  1. +1 −1 venusian/
@@ -353,7 +353,7 @@ def seen(p, m={}):
# don't traverse path items we've seen before
path = [p for p in path if not seen(p)]
- for item in walk_packages(path, name+'.', onerror):
+ for item in walk_packages(path, name+'.', onerror, ignore):
yield item
yield importer, name, ispkg

3 comments on commit 5611bfa

Guys, can we have this patch released?

Without it, there is no way to ignore tests in subpackages - they are getting imported by venusian. And that fails, because tests depends on packages, not available at runtime.


lorenzogil replied Apr 15, 2013

Exactly, I have to add my test dependencies to the install_requires list in order to workaround this bug.


mcdonc replied Apr 15, 2013

1.0a8 is out with the fix.

Please sign in to comment.