rbd: nicer help output for commands without options

Signed-off-by: Mykola Golub <mgolub@suse.com>

diff --git a/src/test/cli/rbd/help.t b/src/test/cli/rbd/help.t
index ec3f4116c5c7cf4694571176a5bf57c81d52b665..c4a0b37f3c6bba32679e1ff88b2d1b428775661c 100644 (file)
--- a/src/test/cli/rbd/help.t
+++ b/src/test/cli/rbd/help.t
@@ -306,8 +306,7 @@
     --image arg          image name
   
   rbd help config pool get
-  usage: rbd config pool get 
-                             <pool-name> <key> 
+  usage: rbd config pool get <pool-name> <key> 
   
   Get a pool-level configuration override.
   
@@ -329,8 +328,7 @@
     --pretty-format      pretty formatting (json and xml)
   
   rbd help config pool remove
-  usage: rbd config pool remove 
-                                <pool-name> <key> 
+  usage: rbd config pool remove <pool-name> <key> 
   
   Remove a pool-level configuration override.
   
@@ -339,8 +337,7 @@
     <key>                config key
   
   rbd help config pool set
-  usage: rbd config pool set 
-                             <pool-name> <key> <value> 
+  usage: rbd config pool set <pool-name> <key> <value> 
   
   Set a pool-level configuration override.
   

diff --git a/src/tools/rbd/OptionPrinter.cc b/src/tools/rbd/OptionPrinter.cc
index 1033b53be703b2e59c6a295622d48e218ab46894..14affb0b3bb85192e9f3d0ac8bbaebb55ee02b24 100644 (file)
--- a/src/tools/rbd/OptionPrinter.cc
+++ b/src/tools/rbd/OptionPrinter.cc
@@ -37,7 +37,10 @@ void OptionPrinter::print_short(std::ostream &os, size_t initial_offset) {
     }
     indent_stream << " ";
   }
-  indent_stream << std::endl;
+
+  if (m_optional.options().size() > 0 || m_positional.options().size() == 0) {
+    indent_stream << std::endl;
+  }
 
   if (m_positional.options().size() > 0) {
     indent_stream.set_delimiter(" ");