netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] ip: Add examples to usage help
@ 2024-02-11 17:57 Yedaya Katsman
  2024-02-11 21:00 ` Stephen Hemminger
  0 siblings, 1 reply; 2+ messages in thread
From: Yedaya Katsman @ 2024-02-11 17:57 UTC (permalink / raw)
  To: netdev; +Cc: Stephen Hemminger, Yedaya Katsman

Currently the usage only shows the syntax with all possible options,
which makes writing the most frequently used commands cumbersome, since it
requires parsing the syntax in your head. This praticularly affects
users new to the command, that resort to reading man pages or search
engines in order to find the correct invocations.

Copy over the examples from the man page, with the same indentation
for the command exaplanations that exist there. I removed the second
link example to save space.

The whole section is indented the same way the other sections in the usage
are, to keep the uniformity.

Signed-off-by: Yedaya Katsman <yedaya.ka@gmail.com>
---
 ip/ip.c | 11 ++++++++++-
 1 file changed, 10 insertions(+), 1 deletion(-)

diff --git a/ip/ip.c b/ip/ip.c
index e51fa206d282..78c3f130c593 100644
--- a/ip/ip.c
+++ b/ip/ip.c
@@ -73,7 +73,16 @@ static void usage(void)
 		"                    -l[oops] { maximum-addr-flush-attempts } | -echo | -br[ief] |\n"
 		"                    -o[neline] | -t[imestamp] | -ts[hort] | -b[atch] [filename] |\n"
 		"                    -rc[vbuf] [size] | -n[etns] name | -N[umeric] | -a[ll] |\n"
-		"                    -c[olor]}\n");
+		"                    -c[olor]}\n"
+		"Examples: ip addr\n"
+		"              Shows addresses assigned to all network interfaces.\n"
+		"          ip neigh\n"
+		"              Shows the current neighbour table in kernel.\n"
+		"          ip link set x up\n"
+		"              Bring up interface x.\n"
+		"          ip route\n"
+		"              Show table routes.\n"
+		);
 	exit(-1);
 }
 
-- 
2.34.1


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] ip: Add examples to usage help
  2024-02-11 17:57 [PATCH] ip: Add examples to usage help Yedaya Katsman
@ 2024-02-11 21:00 ` Stephen Hemminger
  0 siblings, 0 replies; 2+ messages in thread
From: Stephen Hemminger @ 2024-02-11 21:00 UTC (permalink / raw)
  To: Yedaya Katsman; +Cc: netdev

On Sun, 11 Feb 2024 19:57:04 +0200
Yedaya Katsman <yedaya.ka@gmail.com> wrote:

> Currently the usage only shows the syntax with all possible options,
> which makes writing the most frequently used commands cumbersome, since it
> requires parsing the syntax in your head. This praticularly affects
> users new to the command, that resort to reading man pages or search
> engines in order to find the correct invocations.
> 
> Copy over the examples from the man page, with the same indentation
> for the command exaplanations that exist there. I removed the second
> link example to save space.
> 
> The whole section is indented the same way the other sections in the usage
> are, to keep the uniformity.
> 
> Signed-off-by: Yedaya Katsman <yedaya.ka@gmail.com>

No.
Examples belong on man page, not the help message.
Help should be short and concise.

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2024-02-11 21:00 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-02-11 17:57 [PATCH] ip: Add examples to usage help Yedaya Katsman
2024-02-11 21:00 ` Stephen Hemminger

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).