[issue14778] IrrationalVersionError should include the project name

2012-05-14 Thread Éric Araujo
Éric Araujo added the comment: You are right. When the NormalizedVersion class is used standalone, there is no project name to pass, so it is the responsibility of higher-level code (the Metadata class I think here) to raise or log messages including the project name. To make a patch, you c

[issue14778] IrrationalVersionError should include the project name

2012-05-14 Thread Christoph Sieghart
Christoph Sieghart added the comment: I took a look at the code and I don't think that adding an argument to the IrrationalVersionError constructor is the right way to go. The exception is raised in places where we don't know for which project we are raising the exception (eg in packaging.ve

[issue14778] IrrationalVersionError should include the project name

2012-05-10 Thread Éric Araujo
Éric Araujo added the comment: Setting to easy for the next sprints. To do: add an argument to the IrrationalVersion constructor, add a test to make sure the result of __str__ includes it, and adapt the rest of the codebase to pass the project name when the exception is raised. -- k

[issue14778] IrrationalVersionError should include the project name

2012-05-10 Thread Nick Wilson
New submission from Nick Wilson : IrrationalVersionError in packaging/distutils2 should include the name of the project responsible for the error. It currently only includes the version: >>> import packaging.pypi.xmlrpc >>> client = packaging.pypi.xmlrpc.Client() >>> client.search_project