Message122527
Tried to comment in Rietveld but it didn't work for some reason. Anyway, I think the argparse.py patch isn't good - changing the type error message to "'invalid %s value: %r details: "%s"'" will change the behavior of a bunch of programs, and it's not clearly for the better. Instead, you should raise an ArgumentTypeError instead of a ValueError, and give it whatever message you want there. That is, let's keep this patch local to the FileType, and not touch the rest of argparse. |
|
Date |
User |
Action |
Args |
2010-11-27 14:46:44 | bethard | set | recipients:
+ bethard, eric.araujo, doughellmann, SilentGhost, akira, Tarsis.Azevedo |
2010-11-27 14:46:44 | bethard | set | messageid: <1290869204.69.0.430285372206.issue9509@psf.upfronthosting.co.za> |
2010-11-27 14:46:40 | bethard | link | issue9509 messages |
2010-11-27 14:46:40 | bethard | create | |
|