summaryrefslogtreecommitdiffstats
path: root/Lib
diff options
context:
space:
mode:
authorR David Murray <rdmurray@bitdance.com>2012-09-08 16:08:01 (GMT)
committerR David Murray <rdmurray@bitdance.com>2012-09-08 16:08:01 (GMT)
commitb522828d2a6bdc4438441eda837a696851ba4263 (patch)
treec1e6c846d61a8cf6302e5f1498c507bc275a41fa /Lib
parent96efdd422cef75f70770107847b1a97e7e524e4c (diff)
downloadcpython-b522828d2a6bdc4438441eda837a696851ba4263.zip
cpython-b522828d2a6bdc4438441eda837a696851ba4263.tar.gz
cpython-b522828d2a6bdc4438441eda837a696851ba4263.tar.bz2
#15847: allow args to be a tuple in parse_args
This fixes a regression introduced by the fix for issue #13922. Although args is not documented as being allowed to be a tuple, previously this worked and so naturally there are programs in the field that depend on it. Patch by Zbyszek Jędrzejewski-Szmek.
Diffstat (limited to 'Lib')
-rw-r--r--Lib/argparse.py5
-rw-r--r--Lib/test/test_argparse.py18
2 files changed, 22 insertions, 1 deletions
diff --git a/Lib/argparse.py b/Lib/argparse.py
index f77c0c2..52ed3ab 100644
--- a/Lib/argparse.py
+++ b/Lib/argparse.py
@@ -1701,9 +1701,12 @@ class ArgumentParser(_AttributeHolder, _ActionsContainer):
return args
def parse_known_args(self, args=None, namespace=None):
- # args default to the system args
if args is None:
+ # args default to the system args
args = _sys.argv[1:]
+ else:
+ # make sure that args are mutable
+ args = list(args)
# default Namespace built from parser defaults
if namespace is None:
diff --git a/Lib/test/test_argparse.py b/Lib/test/test_argparse.py
index cc05160..a2f9a69 100644
--- a/Lib/test/test_argparse.py
+++ b/Lib/test/test_argparse.py
@@ -4522,6 +4522,24 @@ class TestTypeFunctionCallWithNonStringDefault(TestCase):
class TestParseKnownArgs(TestCase):
+ def test_arguments_tuple(self):
+ parser = argparse.ArgumentParser()
+ parser.parse_args(())
+
+ def test_arguments_list(self):
+ parser = argparse.ArgumentParser()
+ parser.parse_args([])
+
+ def test_arguments_tuple_positional(self):
+ parser = argparse.ArgumentParser()
+ parser.add_argument('x')
+ parser.parse_args(('x',))
+
+ def test_arguments_list_positional(self):
+ parser = argparse.ArgumentParser()
+ parser.add_argument('x')
+ parser.parse_args(['x'])
+
def test_optionals(self):
parser = argparse.ArgumentParser()
parser.add_argument('--foo')