irkerd: Fix -i / --immediate handling, and add a message argument
authorW. Trevor King <wking@tremily.us>
Wed, 12 Mar 2014 16:58:44 +0000 (09:58 -0700)
committerEric S. Raymond <esr@thyrsus.com>
Wed, 12 Mar 2014 17:07:46 +0000 (13:07 -0400)
commit35e07773867f52c3221cea89fa5ac11f042dae97
tree80372beae81d3e2297317ff2e576c7e6ad91518e
parentcc0c63581884c24a4bd2ba5be1b0b00eb3653bd4
irkerd: Fix -i / --immediate handling, and add a message argument

This fixes some problems with -i/--immediate parsing in ccd311c5
(irkerd: Transition from getopt to argparse, 2014-03-06).

> +    parser.add_argument(
> +        '-i', '--immediate', action='store_const', const=True,
> +        help='disconnect after sending each message')

This does not match the old syntax where -i took two arguments (an IRC
URL and a message).

> -    if immediate:
> +    if args.immediate:
>          irker.irc.add_event_handler("quit", lambda _c, _e: sys.exit(0))
>          irker.handle('{"to":"%s","privmsg":"%s"}' % (immediate, arguments[0]), quit_after=True)
>          irker.irc.spin()

immediate should be args.immediate, and arguments[0] needs to be added
as a new (optional) positional argument.

This patch fixes both issues.
irkerd