Remove version command, introduce --version option

Signed-off-by: Nicolas De Loof <nicolas.deloof@gmail.com>

README.md

@@ -365,7 +365,6 @@ Commands:
   uninstall   Uninstall an application
   upgrade     Upgrade an installed application
   validate    Checks the rendered application is syntactically correct
-  version     Print version information
 
 Run 'docker app COMMAND --help' for more information on a command.
 ```

e2e/main_test.go

@@ -120,7 +120,7 @@ func TestMain(m *testing.M) {
 
 	dockerCli = dockerCliCommand{path: dockerCliPath, cliPluginDir: cliPluginDir}
 
-	cmd := exec.Command(dockerApp, "app", "version")
+	cmd := exec.Command(dockerApp, "app", "--version")
 	output, err := cmd.CombinedOutput()
 	if err != nil {
 		panic(err)

e2e/testdata/plugin-usage-experimental.golden

@@ -1,8 +1,11 @@
 
-Usage:	docker app COMMAND
+Usage:	docker app [OPTIONS] COMMAND
 
 A tool to build and manage Docker Applications.
 
+Options:
+      --version   Print version information
+
 Commands:
   bundle      Create a CNAB invocation image and `bundle.json` for the application
   init        Initialize Docker Application definition
@@ -16,6 +19,5 @@ Commands:
   uninstall   Uninstall an application
   upgrade     Upgrade an installed application
   validate    Checks the rendered application is syntactically correct
-  version     Print version information
 
 Run 'docker app COMMAND --help' for more information on a command.

e2e/testdata/plugin-usage.golden

@@ -1,8 +1,11 @@
 
-Usage:	docker app COMMAND
+Usage:	docker app [OPTIONS] COMMAND
 
 A tool to build and manage Docker Applications.
 
+Options:
+      --version   Print version information
+
 Commands:
   bundle      Create a CNAB invocation image and `bundle.json` for the application
   init        Initialize Docker Application definition
@@ -16,6 +19,5 @@ Commands:
   uninstall   Uninstall an application
   upgrade     Upgrade an installed application
   validate    Checks the rendered application is syntactically correct
-  version     Print version information
 
 Run 'docker app COMMAND --help' for more information on a command.

internal/commands/root.go

@@ -2,6 +2,7 @@ package commands
 
 import (
 	"fmt"
+	"github.com/docker/app/internal"
 	"io/ioutil"
 	"os"
 
@@ -14,6 +15,7 @@ import (
 
 var (
 	completion string
+	showversion bool
 )
 
 // NewRootCmd returns the base root command.
@@ -24,6 +26,12 @@ func NewRootCmd(use string, dockerCli command.Cli) *cobra.Command {
 		Use:         use,
 		Annotations: map[string]string{"experimentalCLI": "true"},
 		RunE: func(cmd *cobra.Command, args []string) error {
+
+			if showversion {
+				fmt.Fprintln(os.Stdout, internal.FullVersion()) //nolint:errcheck
+				return nil
+			}
+
 			switch completion {
 			case "bash":
 				return cmd.GenBashCompletion(dockerCli.Out())
@@ -41,10 +49,11 @@ func NewRootCmd(use string, dockerCli command.Cli) *cobra.Command {
 
 	cmd.Flags().StringVar(&completion, "completion", "", "Generates completion scripts for the specified shell (bash or zsh)")
 	if err := cmd.Flags().MarkHidden("completion"); err != nil {
-		_, _ = fmt.Fprintf(os.Stderr, "Failed to register command line options: %v", err.Error())
+		fmt.Fprintf(os.Stderr, "Failed to register command line options: %v", err.Error()) //nolit:errcheck
 		return nil
 	}
 
+	cmd.Flags().BoolVar(&showversion, "version", false,"Print version information")
 	return cmd
 }
 
@@ -59,7 +68,6 @@ func addCommands(cmd *cobra.Command, dockerCli command.Cli) {
 		inspectCmd(dockerCli),
 		renderCmd(dockerCli),
 		validateCmd(),
-		versionCmd(dockerCli),
 		bundleCmd(dockerCli),
 		pushCmd(dockerCli),
 		pullCmd(dockerCli),

internal/version.go

@@ -17,14 +17,13 @@ var (
 )
 
 // FullVersion returns a string of version information.
-func FullVersion(invocationBaseImage string) string {
+func FullVersion() string {
 	res := []string{
 		fmt.Sprintf("Version:               %s", Version),
 		fmt.Sprintf("Git commit:            %s", GitCommit),
 		fmt.Sprintf("Built:                 %s", reformatDate(BuildTime)),
 		fmt.Sprintf("OS/Arch:               %s/%s", runtime.GOOS, runtime.GOARCH),
-		fmt.Sprintf("Experimental:          %s", Experimental),
-		fmt.Sprintf("Invocation Base Image: %s", invocationBaseImage),
+		// +experimental fmt.Sprintf("Experimental:          %s", Experimental),
 	}
 	return strings.Join(res, "\n")
 }