[PATCH 4/9] add command line parsing to daemon
Ray Strode
halfline at gmail.com
Mon Feb 23 13:33:36 PST 2009
Hi,
On Mon, Feb 23, 2009 at 3:35 PM, <william.jon.mccann at gmail.com> wrote:
> From: William Jon McCann <jmccann at redhat.com>
>
> If for no other reason than to handle --help.
Right, makes a lot of sense. I've been wanting to do this for a
while, but never made it to the top of my list.
[...]
> --- a/src/main.c
> +++ b/src/main.c
[...]
> + ply_command_parser_add_options (state.command_parser,
> + "help", "This help message", PLY_COMMAND_OPTION_TYPE_FLAG,
> + "attach-to-session", "pty_master_fd", PLY_COMMAND_OPTION_TYPE_LONG,
> + NULL);
> +
I mentioned this in an earlier thread, but this should be TYPE_INT not TYPE_LONG
Otherwise, looks good! Feel free to push.
--Ray
More information about the plymouth
mailing list