Todd DeLuca <todddel...@gmail.com> added the comment:

This patch fixes the problem where the user would select a maturity status when 
runnning 'pysetup create' and the resulting setup.cfq would have a maturity 
status one less than the user selected.

It also fixes the behavior where a user can select '0' as a maturity status, 
which is not a valid selection, and setup.cfg will contain maturity status 
'Development Status :: 7 - Inactive'.  This behavior is related to the 
implementation of the off-by-one error above, since '0' is translated into a 
list lookup using the index -1.  

Finally, the patch addresses behavior where a user cannot enter a blank 
maturity status (e.g. by just pressing the 'return' key) in order to skip the 
question.

The unit tests added to 'test_create.py' cover expected behavior when entering 
and blank selection, an invalid selection, and a valid selection.  

These tests were run using Python 2.7.3 on my Mac OS X 10.6.8 (Snow Leopard).

----------
keywords: +patch
Added file: http://bugs.python.org/file25731/issue14858.patch

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue14858>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to