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

Justin Love report at bugs.python.org
Sat Sep 24 03:27:50 CEST 2011


Justin Love <python at JustinLove.name> 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 turned out I had to update to even have 'packaging'

Error was caused by an actual change in behavior - I was testing the argument-check by providing an action with no argument.  The new code assumes a single 'word' is a filename, and translates it to include, eg. 'include include' which then passes the parse.

I modified the Manifest to test against the list of actions, since this makes it harder for users to make a mistake.  Of course we could just drop the test, or only check multi-argument actions.

----------
Added file: http://bugs.python.org/file23235/packaging-manifest_v2.diff

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


More information about the Python-bugs-list mailing list