Roundup Robot added the comment:
New changeset 5d8454fcc629 by R David Murray in branch '3.2':
#15847: allow args to be a tuple in parse_args
http://hg.python.org/cpython/rev/5d8454fcc629
New changeset 76655483c5c8 by R David Murray in branch 'default':
merge #15847: allow args to be a tuple in
R. David Murray added the comment:
Thanks, Zbyszek. I'm glad you caught this.
--
resolution: - fixed
stage: - committed/rejected
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15847
Georg Brandl added the comment:
Committed in release clone as 8c2e87aeb707. Please apply to the main branches
as usual.
--
priority: release blocker - high
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15847
New submission from Zbyszek Jędrzejewski-Szmek:
After recent change (78307 '#13922: argparse no longer incorrectly strips '--'
after the first one.'), parse_args stopped working with a tuple
argument. It is easy to pass tuple to argparse by using positional function
arguments:
def f(*args):
R. David Murray added the comment:
I wonder if this is problematic enough that it should be treated as a
regression and fixed in the next RC?
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15847
Zbyszek Jędrzejewski-Szmek added the comment:
On Sun, Sep 02, 2012 at 02:42:34PM +, R. David Murray wrote:
R. David Murray added the comment:
I wonder if this is problematic enough that it should be treated as a
regression and fixed in the next RC?
I believe yes, because I've already
R. David Murray added the comment:
Let's see what Georg thinks.
--
nosy: +benjamin.peterson, georg.brandl
priority: normal - release blocker
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15847
Steven Bethard added the comment:
The fix looks about right to me.
There's a bug in the tests though:
parser.parse_args(('x'))
should probably be:
parser.parse_args(('x',))
since I assume the intent was to test tuples.
--
___
Python
Changes by Zbyszek Jędrzejewski-Szmek zbys...@in.waw.pl:
Added file: http://bugs.python.org/file27103/argparse_parse_args_tuple.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15847
___
Changes by Zbyszek Jędrzejewski-Szmek zbys...@in.waw.pl:
Removed file: http://bugs.python.org/file27095/argparse_parse_args_tuple.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15847
___
Zbyszek Jędrzejewski-Szmek added the comment:
Thanks for noticing. Replaced patch with the ('x') - ('x',) bugfix.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15847
___
Georg Brandl added the comment:
I agree this is a regression and should be fixed.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15847
___
___
12 matches
Mail list logo