[PATCH iproute2 net-next] devlink: Add json and pretty options to help and man

Subsystems: the rest

STALE3434d

3 messages, 3 authors, 2017-03-09 · open the first message on its own page

[PATCH iproute2 net-next] devlink: Add json and pretty options to help and man

From: Roi Dayan <hidden>
Date: 2017-03-06 09:13:05

While at it also fixed missing double dash for long opts.

Signed-off-by: Roi Dayan <redacted>
---
 devlink/devlink.c  |  2 +-
 man/man8/devlink.8 | 14 ++++++++++++--
 2 files changed, 13 insertions(+), 3 deletions(-)
diff --git a/devlink/devlink.c b/devlink/devlink.c
index c357580..e90226e 100644
--- a/devlink/devlink.c
+++ b/devlink/devlink.c
@@ -2470,7 +2470,7 @@ static void help(void)
 {
 	pr_err("Usage: devlink [ OPTIONS ] OBJECT { COMMAND | help }\n"
 	       "where  OBJECT := { dev | port | sb | monitor }\n"
-	       "       OPTIONS := { -V[ersion] | -n[no-nice-names] }\n");
+	       "       OPTIONS := { -V[ersion] | -n[no-nice-names] | -j[json] | -p[pretty] }\n");
 }
 
 static int dl_cmd(struct dl *dl)
diff --git a/man/man8/devlink.8 b/man/man8/devlink.8
index cf0563b..a480766 100644
--- a/man/man8/devlink.8
+++ b/man/man8/devlink.8
@@ -20,19 +20,29 @@ devlink \- Devlink tool
 .IR OPTIONS " := { "
 \fB\-V\fR[\fIersion\fR] |
 \fB\-n\fR[\fIno-nice-names\fR] }
+\fB\-j\fR[\fIjson\fR] }
+\fB\-p\fR[\fIpretty\fR] }
 
 .SH OPTIONS
 
 .TP
-.BR "\-V" , " -Version"
+.BR "\-V" , " --Version"
 Print the version of the
 .B devlink
 utility and exit.
 
 .TP
-.BR "\-n" , " -no-nice-names"
+.BR "\-n" , " --no-nice-names"
 Turn off printing out nice names, for example netdevice ifnames instead of devlink port identification.
 
+.TP
+.BR "\-j" , " --json"
+Generate JSON output.
+
+.TP
+.BR "\-p" , " --pretty"
+When combined with -j generate a pretty JSON output.
+
 .SS
 .I OBJECT
 
-- 
2.7.4

Re: [PATCH iproute2 net-next] devlink: Add json and pretty options to help and man

From: Jiri Pirko <jiri@resnulli.us>
Date: 2017-03-06 10:42:08

Mon, Mar 06, 2017 at 10:06:18AM CET, roid@mellanox.com wrote:
While at it also fixed missing double dash for long opts.

Signed-off-by: Roi Dayan <redacted>
Acked-by: Jiri Pirko <redacted>

Re: [PATCH iproute2 net-next] devlink: Add json and pretty options to help and man

From: Stephen Hemminger <stephen@networkplumber.org>
Date: 2017-03-09 02:00:10

On Mon,  6 Mar 2017 11:06:18 +0200
Roi Dayan [off-list ref] wrote:
While at it also fixed missing double dash for long opts.

Signed-off-by: Roi Dayan <redacted>
Applied to master (for 4.11) this doesn't have to wait until net-next (4.12)
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help