summaryrefslogtreecommitdiff
path: root/Lib/argparse.py
diff options
context:
space:
mode:
authorVinay Sajip <vinay_sajip@yahoo.co.uk>2011-06-09 18:43:11 +0100
committerVinay Sajip <vinay_sajip@yahoo.co.uk>2011-06-09 18:43:11 +0100
commitbf353aadf274b898a515e81c2b1d63fbac83feca (patch)
tree27cc7e55a62ad1d4deab1a16c3f8059f10c7b408 /Lib/argparse.py
parentae5740f139304a2d032602f17b87ec485aa4c750 (diff)
parentf97c59aaba2d93e48cbc6d25f7ff9f9c87f8d0b2 (diff)
downloadcpython-git-bf353aadf274b898a515e81c2b1d63fbac83feca.tar.gz
Merged upstream changes.
Diffstat (limited to 'Lib/argparse.py')
-rw-r--r--Lib/argparse.py15
1 files changed, 5 insertions, 10 deletions
diff --git a/Lib/argparse.py b/Lib/argparse.py
index 0658472acf..f0cfe27eb8 100644
--- a/Lib/argparse.py
+++ b/Lib/argparse.py
@@ -1969,17 +1969,12 @@ class ArgumentParser(_AttributeHolder, _ActionsContainer):
# if we didn't consume all the argument strings, there were extras
extras.extend(arg_strings[stop_index:])
- # if we didn't use all the Positional objects, there were too few
- # arg strings supplied.
- if positionals:
- self.error(_('too few arguments'))
-
# make sure all required actions were present
- for action in self._actions:
- if action.required:
- if action not in seen_actions:
- name = _get_action_name(action)
- self.error(_('argument %s is required') % name)
+ required_actions = [_get_action_name(action) for action in self._actions
+ if action.required and action not in seen_actions]
+ if required_actions:
+ self.error(_('the following arguments are required: %s') %
+ ', '.join(required_actions))
# make sure all required groups had one option present
for group in self._mutually_exclusive_groups: