From 4258e3832627b23927f4a77dd85d1cb78f5a2a65 Mon Sep 17 00:00:00 2001 From: Ed Santiago Date: Thu, 28 Feb 2019 07:29:33 -0700 Subject: Fix usage messages for podman image list, rm pr #2480 fixed the missing 'podman image list/rm' commands; it broke their usage messages. This corrects both usage messages and also their examples. Also: add an e2e test for 'podman image rm' (untested) Signed-off-by: Ed Santiago --- cmd/podman/image.go | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) (limited to 'cmd') diff --git a/cmd/podman/image.go b/cmd/podman/image.go index b5c1c3ccf..0777425eb 100644 --- a/cmd/podman/image.go +++ b/cmd/podman/image.go @@ -1,6 +1,8 @@ package main import ( + "strings" + "github.com/containers/libpod/cmd/podman/cliconfig" "github.com/spf13/cobra" ) @@ -39,11 +41,13 @@ func init() { imageCommand.AddCommand(getImageSubCommands()...) // Setup of "images" to appear as "list" - _imagesSubCommand.Use = "list" + _imagesSubCommand.Use = strings.Replace(_imagesSubCommand.Use, "images", "list", 1) _imagesSubCommand.Aliases = []string{"ls"} + _imagesSubCommand.Example = strings.Replace(_imagesSubCommand.Example, "podman images", "podman image list", -1) imageCommand.AddCommand(&_imagesSubCommand) - // Setup of "rmi" to appears as "rm" - _rmSubCommand.Use = "rm" + // It makes no sense to keep 'podman images rmi'; just use 'rm' + _rmSubCommand.Use = strings.Replace(_rmSubCommand.Use, "rmi", "rm", 1) + _rmSubCommand.Example = strings.Replace(_rmSubCommand.Example, "podman rmi", "podman image rm", -1) imageCommand.AddCommand(&_rmSubCommand) } -- cgit v1.2.3-54-g00ecf