Message137649
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.) |
|
Date |
User |
Action |
Args |
2011-06-04 15:39:04 | eric.araujo | set | recipients:
+ eric.araujo, tarek, srid, alexis |
2011-06-04 15:39:03 | eric.araujo | set | messageid: <1307201943.95.0.121931288732.issue8286@psf.upfronthosting.co.za> |
2011-06-04 15:39:03 | eric.araujo | link | issue8286 messages |
2011-06-04 15:39:03 | eric.araujo | create | |
|