[lttng-dev] [PATCH lttng-tools] Add version info to lttng-relayd help

Jérémie Galarneau jeremie.galarneau at efficios.com
Wed Oct 19 20:57:31 UTC 2016


On 15 October 2016 at 02:38, Anders Wallin <wallinux at gmail.com> wrote:
> lttng-relayd man pages states that the option
> -V --version is available, but it it's missing in the code
>
> Signed-off-by: Anders Wallin <wallinux at gmail.com>
> ---
>  src/bin/lttng-relayd/main.c | 15 ++++++++++++++-
>  1 file changed, 14 insertions(+), 1 deletion(-)
>
> diff --git a/src/bin/lttng-relayd/main.c b/src/bin/lttng-relayd/main.c
> index ea46ec7..34a9ba1 100644
> --- a/src/bin/lttng-relayd/main.c
> +++ b/src/bin/lttng-relayd/main.c
> @@ -161,10 +161,20 @@ static struct option long_options[] = {
>         { "output", 1, 0, 'o', },
>         { "verbose", 0, 0, 'v', },
>         { "config", 1, 0, 'f' },
> +       { "version", 0, 0, 'V' },
>         { NULL, 0, 0, 0, },
>  };
>
> -static const char *config_ignore_options[] = { "help", "config" };
> +static const char *config_ignore_options[] = { "help", "config", "version" };
> +
> +/*
> + * Version info on stderr
> + */
> +static void version(void)
> +{
> +       fprintf(stderr, "%s (LTTng Trace Control) " VERSION" - " VERSION_NAME "\n",
> +               progname);

Please use the short version printout, as done in the session daemon.
Also, please output to stdout.

See:
https://github.com/lttng/lttng-tools/blob/master/src/bin/lttng-sessiond/main.c#L4686

> +}
>
>  /*
>   * Take an option from the getopt output and set it in the right variable to be
> @@ -255,6 +265,9 @@ static int set_option(int opt, const char *arg, const char *optname)
>                         perror("exec");
>                 }
>                 exit(EXIT_FAILURE);
> +       case 'V':
> +               version();
> +               exit(EXIT_FAILURE);

EXIT_FAILURE -> EXIT_SUCCESS

Thanks,
Jérémie

>         case 'o':
>                 if (lttng_is_setuid_setgid()) {
>                         WARN("Getting '%s' argument from setuid/setgid binary refused for security reasons.",
> --
> 2.10.0
>
> _______________________________________________
> lttng-dev mailing list
> lttng-dev at lists.lttng.org
> https://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev



-- 
Jérémie Galarneau
EfficiOS Inc.
http://www.efficios.com


More information about the lttng-dev mailing list