homepage

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_projects('req')
 Irrational version error found: 0.0.0-prealpha
 Irrational version error found: 0.1dev-20110502
 ...

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.

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.version.NormalizedVersion).
Wouldn't it be better to make packaging.pypi.xmlrpc.Client just print the
package name with the exception?
I would be glad to contribute a patch.

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 can use the hg.python.org/distutils2 repo (also mirrored on Bitbucket under the python_mirrors user account for easing cloning) or the CPython repository. Thanks!

BTW there could be an optional argument in the exception constructor for the project name, but that does not seem clean to me.

The optional argument approach doesn't seem clean to me either. 
I think the correct approach is - as you wrote - to change the higher level code to include the project name in the exception message.

I wrote a trivial patch against the CPython repository that adds the name to the exception message. Did I forget anything? What kind of test would make sense?

distutils2 development has stopped.