[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