diff options
author | openshift-ci[bot] <75433959+openshift-ci[bot]@users.noreply.github.com> | 2022-06-16 18:06:29 +0000 |
---|---|---|
committer | GitHub <noreply@github.com> | 2022-06-16 18:06:29 +0000 |
commit | 5cba4dc2e490b5c652ba0821067b63d7b2e4cf8f (patch) | |
tree | 5f39931a2eb7821f992ae849a7ab5f57607a2784 /cmd | |
parent | 8765adb756c570a13f06b9c60596d34567d6568b (diff) | |
parent | 09c462d735082efa0489b66c66f0590460f4f00c (diff) | |
download | podman-5cba4dc2e490b5c652ba0821067b63d7b2e4cf8f.tar.gz podman-5cba4dc2e490b5c652ba0821067b63d7b2e4cf8f.tar.bz2 podman-5cba4dc2e490b5c652ba0821067b63d7b2e4cf8f.zip |
Merge pull request #14619 from Luap99/help
fix "podman -h" help output
Diffstat (limited to 'cmd')
-rw-r--r-- | cmd/podman/main.go | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/cmd/podman/main.go b/cmd/podman/main.go index 929c8a757..c6ba69e94 100644 --- a/cmd/podman/main.go +++ b/cmd/podman/main.go @@ -1,6 +1,7 @@ package main import ( + "errors" "fmt" "os" @@ -26,6 +27,7 @@ import ( "github.com/containers/storage/pkg/reexec" "github.com/sirupsen/logrus" "github.com/spf13/cobra" + "github.com/spf13/pflag" ) func main() { @@ -101,6 +103,13 @@ func parseCommands() *cobra.Command { } func flagErrorFuncfunc(c *cobra.Command, e error) error { + // cobra compares via == and not errors.Is so we cannot wrap that error. + // This is required to make podman -h work. + // This can be removed once https://github.com/spf13/cobra/pull/1730 + // is merged and vendored into podman. + if errors.Is(e, pflag.ErrHelp) { + return e + } e = fmt.Errorf("%w\nSee '%s --help'", e, c.CommandPath()) return e } |