[issue9571] argparse: Allow the use of -- to break out of nargs and into subparser
paul j3 added the comment:
This patch modifies that '--' handling from 13922, removing '--' from PARSER
and REMAINDER if it is the first string:
if action.nargs not in [PARSER, REMAINDER]:
try:
arg_strings.remove('--')
except ValueError:
pass
else:
# remove '--' if it is the first string, issue9571
if arg_strings and arg_strings[0] == '--':
arg_strings = arg_strings[1:]
Doing this for PARSER addresses this issue, and should not have any backward
compatibility issues, since starting a 'A...' list with '--' does not make
sense.
The argument for doing this with REMAINDER is a bit weaker. I can't think of
why a user would expect or want an initial '--', but someone might have written
code to compensate for this flaw.
test_argparse has 2 new tests, added at the end of the subparsers class. The
one for subparser has to contend with a bug that makes subparsers optional
(http://bugs.python.org/issue9253). The REMAINDER one is clearly related,
though it doesn't fit the test class theme.
--
keywords: +patch
Added file: http://bugs.python.org/file35886/issue9571_1.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue9571
___
___
Python-bugs-list mailing list
Unsubscribe:
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue9571] argparse: Allow the use of -- to break out of nargs and into subparser
paul j3 added the comment: In elsdoerfer's example, the '--' effectively ends the argument list for '--ignore'. '--' are not allowed in the arguments of an optional (see the end of '_get_nargs_pattern()'). Rather the problem is at the start of _get_values() if action.nargs not in [PARSER, REMAINDER]: arg_strings = [s for s in arg_strings if s != '--'] '--' is not stripped out of the 'parser' input, hence the error message: error: invalid choice: '--' You can see this by replacing the subparsers with an argument with 'nargs=PARSER'. The argument will get `['--','COMMAND',...]`. http://bugs.python.org/issue13922 tries to rework how '--' are handled. Ideally only the current '--' should be removed, leaving the rest to be handled by the subparser (or whoever else gets the strings). Looks like the 13922 fix needs another fix, one that removes '--' if it is the 1st string for a REMAINDER or PARSER argument. With that fix: ./script.py --ignore one two -- COMMAND arg1 arg2 should work, assigning ['one','two'] to 'ignore', and ['arg1','arg2'] to 'COMMAND's positional. (I've tested this in a development version with many other changes. I'll try to write a simpler patch.) If a 'end of a list' flag is still needed (as between 2 * positionals), a 'counter' or 'store_true' optional could be used. Or a new action class that doesn't write anything to the namespace could be written. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue9571 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue9571] argparse: Allow the use of -- to break out of nargs and into subparser
Changes by Mark Lawrence breamore...@yahoo.co.uk: -- nosy: +paul.j3 versions: +Python 3.5 -Python 3.2 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue9571 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue9571] argparse: Allow the use of -- to break out of nargs and into subparser
Changes by Éric Araujo mer...@netwok.org: -- nosy: +eric.araujo ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue9571 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue9571] argparse: Allow the use of -- to break out of nargs and into subparser
Steven Bethard steven.beth...@gmail.com added the comment: This is closely related to issue 9338. The parser should know that your command line requires at least the COMMAND argument, so it should stop parsing in time for that. However, in the case of subcommands, even if we solved issue 9338, you would still get the behavior that ./script.py --ignore one two COMMAND arg1 arg2 would get parsed as arg2 being the command. So I guess there still ought to be a way to tell argparse to stop parsing nargs='+' optionals. Seems like there's also a bug in the current behavior - you should get an error saying that no command was given, not an error saying you issued the command --. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue9571 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue9571] argparse: Allow the use of -- to break out of nargs and into subparser
New submission from Michael.Elsdörfer mich...@elsdoerfer.info: argparse already seems to support -- to indicate that what follows are positional arguments. However, I would like to parse something like: ./script.py --ignore one two -- COMMAND I.e., --ignore is an nargs='+' argument, and I need a way to break out of --ignore and have argparse consider what follows on it's own merits. If COMMAND in the above example refers to a subparser, this won't work: error: invalid choice: '--' (choose from 'command1', 'command2', 'command3') I'm not sure what's the best solution here. Allowing -- here would change the semantics of forcing everything that follows to be positional arguments, since the subparser might have flags. I'm not sure if that is what is required by Unix conventions, but if not, then I think it makes sense to allow -- to be followed by a subparser. -- components: Library (Lib) messages: 113616 nosy: elsdoerfer priority: normal severity: normal status: open title: argparse: Allow the use of -- to break out of nargs and into subparser type: behavior versions: Python 2.7, Python 3.1, Python 3.2, Python 3.3 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue9571 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue9571] argparse: Allow the use of -- to break out of nargs and into subparser
R. David Murray rdmur...@bitdance.com added the comment: It looks like, if accepted, this would be a feature request,so I'm marking it as such and setting versions to 3.2. You'd have to provide a patch pretty soon to get it in to 3.2, though. However, I'm guessing that this is something better off implemented via subclassing in your code, since it sounds like a fairly unusual command pattern. (I've only ever seen options after a -- in a unix comand when those options were being passed unparsed to some *other* command that the first command was a wrapper for.) I've added Steven as nosy, we'll see what he thinks. -- nosy: +bethard, r.david.murray type: behavior - feature request versions: -Python 2.7, Python 3.1, Python 3.3 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue9571 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
