From 0c3192da8c0686b1fe6aba393c1d3279e41c48a0 Mon Sep 17 00:00:00 2001 From: Sebastiaan van Stijn Date: Sat, 19 May 2018 03:15:08 +0200 Subject: [PATCH] Use Cobra built-in --version feature Signed-off-by: Sebastiaan van Stijn --- cli/cobra.go | 1 + cmd/dockerd/docker.go | 11 ++--------- cmd/dockerd/options.go | 1 - 3 files changed, 3 insertions(+), 10 deletions(-) diff --git a/cli/cobra.go b/cli/cobra.go index f110ceae0a..30854a0092 100644 --- a/cli/cobra.go +++ b/cli/cobra.go @@ -22,6 +22,7 @@ func SetupRootCommand(rootCmd *cobra.Command) { rootCmd.SetHelpTemplate(helpTemplate) rootCmd.SetFlagErrorFunc(FlagErrorFunc) rootCmd.SetHelpCommand(helpCommand) + rootCmd.SetVersionTemplate("Docker version {{.Version}}\n") rootCmd.PersistentFlags().BoolP("help", "h", false, "Print usage") rootCmd.PersistentFlags().MarkShorthandDeprecated("help", "please use --help") diff --git a/cmd/dockerd/docker.go b/cmd/dockerd/docker.go index 850a8a4d6a..463482e938 100644 --- a/cmd/dockerd/docker.go +++ b/cmd/dockerd/docker.go @@ -24,19 +24,16 @@ func newDaemonCommand() *cobra.Command { SilenceErrors: true, Args: cli.NoArgs, RunE: func(cmd *cobra.Command, args []string) error { - if opts.version { - showVersion() - return nil - } opts.flags = cmd.Flags() return runDaemon(opts) }, DisableFlagsInUseLine: true, + Version: fmt.Sprintf("%s, build %s", dockerversion.Version, dockerversion.GitCommit), } cli.SetupRootCommand(cmd) flags := cmd.Flags() - flags.BoolVarP(&opts.version, "version", "v", false, "Print version information and quit") + flags.BoolP("version", "v", false, "Print version information and quit") flags.StringVar(&opts.configFile, "config-file", defaultDaemonConfigFile, "Daemon configuration file") opts.InstallFlags(flags) installConfigFlags(opts.daemonConfig, flags) @@ -45,10 +42,6 @@ func newDaemonCommand() *cobra.Command { return cmd } -func showVersion() { - fmt.Printf("Docker version %s, build %s\n", dockerversion.Version, dockerversion.GitCommit) -} - func main() { if reexec.Init() { return diff --git a/cmd/dockerd/options.go b/cmd/dockerd/options.go index 4fd597d610..a6276add59 100644 --- a/cmd/dockerd/options.go +++ b/cmd/dockerd/options.go @@ -30,7 +30,6 @@ var ( ) type daemonOptions struct { - version bool configFile string daemonConfig *config.Config flags *pflag.FlagSet