[issue12776] argparse: type conversion function should be called only once

2012-07-21 Thread Steven Bethard
Steven Bethard added the comment: The patch looks good to me. I've updated it for trunk and to include Mike Meyer's additional test. All argparse tests pass. Anyone who's able to commit and backport, please do. (I should be able to commit myself, but it's now been too long and my SSH key see

[issue12014] str.format parses replacement field incorrectly

2012-07-21 Thread Ben Wolfson
Ben Wolfson added the comment: Ping! -- ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org

[issue12353] argparse cannot handle empty arguments

2012-07-21 Thread Steven Bethard
Steven Bethard added the comment: Yes, the original patch looks fine to me. I applied and tested it, and it works as expected. Please go ahead and apply. -- ___ Python tracker ___

[issue13966] Add disable_interspersed_args() to argparse.ArgumentParser

2012-07-21 Thread Steven Bethard
Steven Bethard added the comment: The argparse changes and tests look good. The new method needs to be documented. You can see some other things (e.g. Misc/NEWS) that also need to be updated by running "make patchcheck" as described here: http://docs.python.org/devguide/patch.html --

[issue15415] Add temp_dir() and change_cwd() to test.support

2012-07-21 Thread Chris Jerdonek
Chris Jerdonek added the comment: Attaching a patch with tests. -- keywords: +patch stage: -> patch review Added file: http://bugs.python.org/file26468/issue-15415-1.patch ___ Python tracker _

[issue15416] 3 * [] gives a list of 3 cross-referenced lists, a[1]='blash

2012-07-21 Thread Martin Mokrejs
New submission from Martin Mokrejs : Hi, I thought that I can easily create a list of, say 3, nested lists: $ python Python 2.7.3 (default, May 17 2012, 21:10:41) [GCC 4.5.3] on linux2 Type "help", "copyright", "credits" or "license" for more information. >>> 3 * [[]] [[], [], []] >>> a=3 * [

[issue15416] 3 * [] gives a list of 3 cross-referenced lists, a[1]='blah' writes in to ALL three!

2012-07-21 Thread Martin Mokrejs
Changes by Martin Mokrejs : -- title: 3 * [] gives a list of 3 cross-referenced lists, a[1]='blash -> 3 * [] gives a list of 3 cross-referenced lists, a[1]='blah' writes in to ALL three! ___ Python tracker ___

[issue11807] Documentation of add_subparsers lacks information about parametres

2012-07-21 Thread Steven Bethard
Steven Bethard added the comment: Looks good. There was one typo in "parametres" that I've fixed. Should be ready to apply. -- Added file: http://bugs.python.org/file26469/11807_3.patch ___ Python tracker ___

[issue15415] Add temp_dir() and change_cwd() to test.support

2012-07-21 Thread Chris Jerdonek
Chris Jerdonek added the comment: Fixing patch so it will apply cleanly. -- Added file: http://bugs.python.org/file26470/issue-15415-2.patch ___ Python tracker ___ _

[issue13818] argparse: -h listening required options under optional arguments

2012-07-21 Thread Steven Bethard
Changes by Steven Bethard : -- resolution: -> duplicate status: open -> closed superseder: -> argparse: Default Help Message Lists Required Args As Optional ___ Python tracker

[issue15269] Document dircmp.left and dircmp.right

2012-07-21 Thread Senthil Kumaran
Senthil Kumaran added the comment: Given that we have self.left_list and self.left_only ( and self.right_list and self.right_only), I am not sure how adding self.left/self.right is going to add more meaning? It would simply point to the dir1 and dir2 arguments that are being passed. Also, i

[issue15416] 3 * [] gives a list of 3 cross-referenced lists, a[1]='blah' writes in to ALL three!

2012-07-21 Thread Amaury Forgeot d'Arc
Amaury Forgeot d'Arc added the comment: The * operator does not create copies, it duplicates references to existing objects. Please read: http://docs.python.org/faq/programming.html#how-do-i-create-a-multidimensional-list -- nosy: +amaury.forgeotdarc resolution: -> invalid status: op

[issue9625] argparse: Problem with defaults for variable nargs when using choices

2012-07-21 Thread Steven Bethard
Steven Bethard added the comment: I agree that this looks like a bug. I think the fix is something like the attached patch, but it needs some tests to make sure that it fixes your problem. -- keywords: +patch Added file: http://bugs.python.org/file26471/issue9625.diff

[issue14856] argparse: creating an already defined subparsers does not raises an exception

2012-07-21 Thread Steven Bethard
Steven Bethard added the comment: Yeah, overwriting the existing parser is probably not typically what the user intended. However, I could see someone doing this if, say, they had a parser designed by another module writer, and they wanted to use it but just change one of the sub-parsers or

[issue15269] Document dircmp.left and dircmp.right

2012-07-21 Thread Chris Jerdonek
Chris Jerdonek added the comment: > Given that we have self.left_list and self.left_only ( and self.right_list > and self.right_only), I am not sure how adding self.left/self.right is going > to add more meaning? It adds more meaning because you can't construct self.left and self.right from

[issue15415] Add temp_dir() and change_cwd() to test.support

2012-07-21 Thread Chris Jerdonek
Chris Jerdonek added the comment: Trying again to have the patch apply cleanly (for the review link). -- Added file: http://bugs.python.org/file26472/issue-15415-3.patch ___ Python tracker

[issue15250] document that filecmp.dircmp comparisons are "shallow"

2012-07-21 Thread Senthil Kumaran
Senthil Kumaran added the comment: Thanks for this report. I saw that filecmp.rst could be improved a bit further too. I added some additional improvements to the docs in addition to the one you mentioned in the patch. -- assignee: docs@python -> orsenthil nosy: +orsenthil Added file:

[issue15417] Add support for csh and fish in venv activation scripts

2012-07-21 Thread Andrew Svetlov
Changes by Andrew Svetlov : -- components: Library (Lib) nosy: asvetlov priority: normal severity: normal status: open title: Add support for csh and fish in venv activation scripts versions: Python 3.3 ___ Python tracker

[issue15417] Add support for csh and fish in venv activation scripts

2012-07-21 Thread Andrew Svetlov
New submission from Andrew Svetlov : I have added required shell files -- keywords: +needs review, patch Added file: http://bugs.python.org/file26474/issue15417.diff ___ Python tracker _

[issue15417] Add support for csh and fish in venv activation scripts

2012-07-21 Thread Andrew Svetlov
Andrew Svetlov added the comment: No idea if Doc/using/venv-create.inc should be updated to reflect support of new shells. virtualenv does nothing in own documentation btw. -- ___ Python tracker ___

[issue14392] type=bool doesn't raise error in argparse.Action

2012-07-21 Thread Toshihiro Kamishima
Toshihiro Kamishima added the comment: > - type -- The type which the command-line arguments should be converted > to, should be one of 'string', 'int', 'float', 'complex' or a > callable object that accepts a single string argument. If None, > 'string' is assumed. I

[issue15250] document that filecmp.dircmp comparisons are "shallow"

2012-07-21 Thread Chris Jerdonek
Chris Jerdonek added the comment: For attribution purposes and to keep the issue focused, shouldn't those other changes be added as part of a separate issue? The changes you added are much larger than mine, and address cosmetic issues and other issues unrelated to the original point about dir

[issue15417] Add support for csh and fish in venv activation scripts

2012-07-21 Thread Andrew Svetlov
Changes by Andrew Svetlov : -- nosy: +carljm, vinay.sajip ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http:

[issue14578] importlib doesn't check Windows registry for paths

2012-07-21 Thread Brian Curtin
Brian Curtin added the comment: Can someone tell me where you want this changed? I guess Lib/imp.py in the find_module function, based on previous mention of imp.find_module? That's shown as deprecated so I'm hesitant to start there. I don't really know anything about importlib, especially no

[issue15250] document that filecmp.dircmp comparisons are "shallow"

2012-07-21 Thread Senthil Kumaran
Senthil Kumaran added the comment: If by attribution, if you mean, proper mention of credits/ acks, during the commit, I shall add it. Also, I agree that "shallow" should be documented. But there were few more places which were lacking, so I ended up with expanding docs after reading the code. I

[issue12353] argparse cannot handle empty arguments

2012-07-21 Thread Roundup Robot
Roundup Robot added the comment: New changeset ac53876d1cc8 by R David Murray in branch '3.2': #12353: argparse now correctly handles null argument values. http://hg.python.org/cpython/rev/ac53876d1cc8 New changeset c4ad8a6eb0df by R David Murray in branch 'default': Merge #12353: argparse now

[issue12353] argparse cannot handle empty arguments

2012-07-21 Thread R. David Murray
R. David Murray added the comment: Done. -- resolution: -> fixed stage: test needed -> committed/rejected status: open -> closed ___ Python tracker ___

[issue14489] repr() function link on the built-in function documentation is incorrect

2012-07-21 Thread H Xu
H Xu added the comment: I think this has been fixed. Where did you see the incorrect link? -- ___ Python tracker ___ ___ Python-bugs-

[issue13922] argparse handling multiple "--" in args improperly

2012-07-21 Thread Roundup Robot
Roundup Robot added the comment: New changeset 18b114be013e by R David Murray in branch '3.2': #13922: argparse no longer incorrectly strips '--' after the first one. http://hg.python.org/cpython/rev/18b114be013e New changeset bd2c167dfabc by R David Murray in branch '2.7': #13922: argparse no

[issue13922] argparse handling multiple "--" in args improperly

2012-07-21 Thread R. David Murray
R. David Murray added the comment: Committed. Thanks, Jeff. By the way, although this patch isn't big enough to require it, it would be great if you would submit a contributor agreement: http://www.python.org/psf/contrib. -- resolution: -> fixed stage: -> committed/rejected status

[issue13720] argparse print_help() fails if COLUMNS is set to a low value

2012-07-21 Thread Russell Sim
Russell Sim added the comment: Hi, I am having the same problem while running ipython in a batch mode emacs. Apparently you can't even start ipython if the columns are less than 27, since they use the argparse library for the magic method help printing and they preformat the strings at impo

[issue13922] argparse handling multiple "--" in args improperly

2012-07-21 Thread Warren Turkal
Warren Turkal added the comment: Thanks for fixing this issue. You guys are great! wt On Sat, Jul 21, 2012 at 8:00 PM, R. David Murray wrote: > > R. David Murray added the comment: > > Committed. Thanks, Jeff. By the way, although this patch isn't big > enough to require it, it would be gr

[issue15184] Test failure in test_sysconfig_module

2012-07-21 Thread Roundup Robot
Roundup Robot added the comment: New changeset 31349bc40214 by Ned Deily in branch 'default': Issue #15184: Fix test__remove_unsupported_archs failures on 10.6 http://hg.python.org/cpython/rev/31349bc40214 -- status: pending -> open ___ Python tracke

<    1   2