[issue11751] Increase distutils.filelist / packaging.manifest test coverage

Éric Araujo report at bugs.python.org
Sun Sep 25 03:54:20 CEST 2011


Éric Araujo <merwok at netwok.org> added the comment:

> Sorry, it's been sitting at the bottom of my starred list for months,
> the prospect of reloading context always making it a little less
> attractive than something else.
I understand that very well :)

> Error was caused by an actual change in behavior
Ah, good, I had missed that change.  I’ll just adapt the test to check that one-word == include.

I think I won’t add the Manifest._actions attribute as in your patch; it does not cost much to have the list once in manifest and another time in test_manifest.

The only change I’ll make (or maybe you’d like to do it?) before committing is to add a test to make sure the line marked with “this cannot happen” really cannot happen.

----------

_______________________________________
Python tracker <report at bugs.python.org>
<http://bugs.python.org/issue11751>
_______________________________________


More information about the Python-bugs-list mailing list