diff options
author | Lennart Poettering <lennart@poettering.net> | 2018-08-09 10:32:31 +0200 |
---|---|---|
committer | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2018-08-20 11:33:04 +0200 |
commit | 37ec0fdd3443a77a5120cf55002fedcb5b1dd069 (patch) | |
tree | 75c684fd19b60094f5a89220c728f5e12e4c34b8 /src/sysctl/sysctl.c | |
parent | util: improve comments why we ignore EACCES and EPERM (diff) | |
download | systemd-37ec0fdd3443a77a5120cf55002fedcb5b1dd069.tar.xz systemd-37ec0fdd3443a77a5120cf55002fedcb5b1dd069.zip |
tree-wide: add clickable man page link to all --help texts
This is a bit like the info link in most of GNU's --help texts, but we
don't do info but man pages, and we make them properly clickable on
terminal supporting that, because awesome.
I think it's generally advisable to link up our (brief) --help texts and
our (more comprehensive) man pages a bit, so this should be an easy and
straight-forward way to do it.
Diffstat (limited to 'src/sysctl/sysctl.c')
-rw-r--r-- | src/sysctl/sysctl.c | 19 |
1 files changed, 15 insertions, 4 deletions
diff --git a/src/sysctl/sysctl.c b/src/sysctl/sysctl.c index 0151f7dabe..945ae3746b 100644 --- a/src/sysctl/sysctl.c +++ b/src/sysctl/sysctl.c @@ -160,7 +160,14 @@ static int parse_file(OrderedHashmap *sysctl_options, const char *path, bool ign return r; } -static void help(void) { +static int help(void) { + _cleanup_free_ char *link = NULL; + int r; + + r = terminal_urlify_man("systemd-sysctl.service", "8", &link); + if (r < 0) + return log_oom(); + printf("%s [OPTIONS...] [CONFIGURATION FILE...]\n\n" "Applies kernel sysctl settings.\n\n" " -h --help Show this help\n" @@ -168,7 +175,12 @@ static void help(void) { " --cat-config Show configuration files\n" " --prefix=PATH Only apply rules with the specified prefix\n" " --no-pager Do not pipe output into a pager\n" - , program_invocation_short_name); + "\nSee the %s for details.\n" + , program_invocation_short_name + , link + ); + + return 0; } static int parse_argv(int argc, char *argv[]) { @@ -199,8 +211,7 @@ static int parse_argv(int argc, char *argv[]) { switch (c) { case 'h': - help(); - return 0; + return help(); case ARG_VERSION: return version(); |