Issue 9306: distutils: raise informative error message when cmd_class is None (original) (raw)

Issue9306

Created on 2010-07-19 16:35 by srid, last changed 2022-04-11 14:57 by admin. This issue is now closed.

Messages (3)
msg110776 - (view)	Author: Sridhar Ratnakumar (srid)	Date: 2010-07-19 16:35
Current I see this: File "/opt/ActivePython-2.7/lib/python2.7/distutils/dist.py", line 806, in get_command_class klass = self.cmdclass.get(command) AttributeError: 'NoneType' object has no attribute 'get' more context: https://bugs.launchpad.net/pyreadline/+bug/607329
msg131591 - (view)	Author: Éric Araujo (eric.araujo) *	Date: 2011-03-21 01:13
I’m inclined to reject the bug: http://docs.python.org/distutils/apiref#distutils.core.setup clearly says that the argument should be a dictionary. The code in pyreadline should set it to an empty dict instead of None. While it can be argued that providing clearly helpful messages is a service for our users, especially given that people writing setup scripts may be beginners, it seems to me that adding type checking would not be the Python way. Raymond, what do you think?
msg144460 - (view)	Author: Éric Araujo (eric.araujo) *	Date: 2011-09-23 16:55
Given that it is documented that the argument must be a dictionary, I’m rejecting this.