voussoir/cmd
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Fix behavior of --all being both positional and named arg.

Browse source
This commit is contained in:
voussoir 2020-11-20 01:37:21 -08:00
parent 482c787f79
commit 844dc5894b
1 changed files with 9 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
search.py
View file

@ -220,7 +220,7 @@ def argparse_to_dict(args):
content_args = None content_args = None
return { return {
'yes_all': args.yes_all, 'yes_all': args.yes_all_1 + args.yes_all_2,
'yes_any': args.yes_any, 'yes_any': args.yes_any,
'not_all': args.not_all, 'not_all': args.not_all,
'not_any': args.not_any, 'not_any': args.not_any,
@ -267,11 +267,14 @@ def main(argv):
name_args.pop(0) name_args.pop(0)
content_args = [item for chunk in halves[2:] for item in chunk] content_args = [item for chunk in halves[2:] for item in chunk]
parser.add_argument('yes_all', nargs='*', default=None) # argparse doesn't work well when an argument is both positional and
parser.add_argument('--all', dest='yes_all', nargs='+') # named, so both of these yes_all will be combined into a single list
parser.add_argument('--any', dest='yes_any', nargs='+') # during the gateway function.
parser.add_argument('--not_all', '--not-all', dest='not_all', nargs='+') parser.add_argument('yes_all_1', nargs='*', default=None)
parser.add_argument('--not_any', '--not-any', dest='not_any', nargs='+') parser.add_argument('--all', dest='yes_all_2', nargs='+', default=[])
parser.add_argument('--any', dest='yes_any', nargs='+', default=[])
parser.add_argument('--not_all', '--not-all', dest='not_all', nargs='+', default=[])
parser.add_argument('--not_any', '--not-any', dest='not_any', nargs='+', default=[])
parser.add_argument('--strip', dest='do_strip', action='store_true') parser.add_argument('--strip', dest='do_strip', action='store_true')
parser.add_argument('--case', dest='case_sensitive', action='store_true') parser.add_argument('--case', dest='case_sensitive', action='store_true')