Use spf13/cobra for docker commit

This fix is part of the effort to convert commands to spf13/cobra #23211.

Thif fix coverted command `docker commit` to use spf13/cobra

NOTE: `RequiresMinMaxArgs()` has been renamed to `RequiresRangeArgs()`.

Signed-off-by: Yong Tang <yong.tang.github@outlook.com>
This commit is contained in:
Yong Tang 2016-06-06 07:57:44 -07:00
parent 58a97cc11e
commit 43fdd6f2fa
3 changed files with 3 additions and 3 deletions

View File

@ -36,6 +36,7 @@ func NewCobraAdaptor(clientFlags *cliflags.ClientFlags) CobraAdaptor {
rootCmd.SetOutput(stdout) rootCmd.SetOutput(stdout)
rootCmd.AddCommand( rootCmd.AddCommand(
container.NewAttachCommand(dockerCli), container.NewAttachCommand(dockerCli),
container.NewCommitCommand(dockerCli),
container.NewCreateCommand(dockerCli), container.NewCreateCommand(dockerCli),
container.NewDiffCommand(dockerCli), container.NewDiffCommand(dockerCli),
container.NewExportCommand(dockerCli), container.NewExportCommand(dockerCli),

View File

@ -60,8 +60,8 @@ func RequiresMaxArgs(max int) cobra.PositionalArgs {
} }
} }
// RequiresMinMaxArgs returns an error if there is not at least min args and at most max args // RequiresRangeArgs returns an error if there is not at least min args and at most max args
func RequiresMinMaxArgs(min int, max int) cobra.PositionalArgs { func RequiresRangeArgs(min int, max int) cobra.PositionalArgs {
return func(cmd *cobra.Command, args []string) error { return func(cmd *cobra.Command, args []string) error {
if len(args) >= min && len(args) <= max { if len(args) >= min && len(args) <= max {
return nil return nil

View File

@ -8,7 +8,6 @@ type Command struct {
// DockerCommandUsage lists the top level docker commands and their short usage // DockerCommandUsage lists the top level docker commands and their short usage
var DockerCommandUsage = []Command{ var DockerCommandUsage = []Command{
{"commit", "Create a new image from a container's changes"},
{"cp", "Copy files/folders between a container and the local filesystem"}, {"cp", "Copy files/folders between a container and the local filesystem"},
{"exec", "Run a command in a running container"}, {"exec", "Run a command in a running container"},
{"info", "Display system-wide information"}, {"info", "Display system-wide information"},