diff options
author | baude <bbaude@redhat.com> | 2019-02-20 13:46:41 -0600 |
---|---|---|
committer | baude <bbaude@redhat.com> | 2019-02-22 14:13:08 -0600 |
commit | 328250e7bef56b0b69dbea124c1170abd289900f (patch) | |
tree | 03a9d601d3f3ce0ac0d04fc3b2841b970031ed48 /cmd/podman/rm.go | |
parent | 8039ccf4d560acbc308066cf8996bc4e6167a4a9 (diff) | |
download | podman-328250e7bef56b0b69dbea124c1170abd289900f.tar.gz podman-328250e7bef56b0b69dbea124c1170abd289900f.tar.bz2 podman-328250e7bef56b0b69dbea124c1170abd289900f.zip |
Improve command line validation
Use the checkallandlatest function to validate flag usage as part
of the cobra command args validation.
Signed-off-by: baude <bbaude@redhat.com>
Diffstat (limited to 'cmd/podman/rm.go')
-rw-r--r-- | cmd/podman/rm.go | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/cmd/podman/rm.go b/cmd/podman/rm.go index 2e5fe1dc0..8ff2ee727 100644 --- a/cmd/podman/rm.go +++ b/cmd/podman/rm.go @@ -28,6 +28,9 @@ Running containers will not be removed without the -f option. rmCommand.GlobalFlags = MainGlobalOpts return rmCmd(&rmCommand) }, + Args: func(cmd *cobra.Command, args []string) error { + return checkAllAndLatest(cmd, args, false) + }, Example: `podman rm imageID podman rm mywebserver myflaskserver 860a4b23 podman rm --force --all`, @@ -57,10 +60,6 @@ func rmCmd(c *cliconfig.RmValues) error { } defer runtime.Shutdown(false) - if err := checkAllAndLatest(&c.PodmanCommand); err != nil { - return err - } - delContainers, err := getAllOrLatestContainers(&c.PodmanCommand, runtime, -1, "all") if err != nil { if c.Force && len(c.InputArgs) > 0 { |