[issue8286] distutils: path '[...]' cannot end with '/' -- need better error message

[Éric Araujo]

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

There is a simple way to fix this: change one line in sdist to catch ValueErrors in addition to DistutilsTemplateError.  Users will get a on-line warning message with the ill-formed manifest line number instead of a wall of traceback.

This is clearly a bug fix IMO: it is documented that distutils code can raise standard exceptions in addition to Distutils* errors, so here it’s not a new feature but a fix for sdist.

(Another way to fix this, would be to catch ValueError raised by convert_path in filelist and wrap it in a DistutilsTemplateError, but I’m not sure it really buys us anything.  Maybe it would be the better fix for Packaging, where the Manifest class is intended for public use.)

----------
assignee: tarek -> eric.araujo
components: +Distutils2
nosy: +alexis
versions: +Python 2.7, Python 3.1

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