cli/command/image: use opts.PlatformSlice for image remove

Signed-off-by: Sebastiaan van Stijn <[email protected]>

Author: thaJeztah

cli/command/image/remove.go

@@ -6,18 +6,19 @@ import (
 	"fmt"
 
 	cerrdefs "github.com/containerd/errdefs"
-	"github.com/containerd/platforms"
 	"github.com/docker/cli/cli"
 	"github.com/docker/cli/cli/command"
 	"github.com/docker/cli/cli/command/completion"
+	"github.com/docker/cli/opts"
 	"github.com/docker/docker/api/types/image"
+	ocispec "github.com/opencontainers/image-spec/specs-go/v1"
 	"github.com/spf13/cobra"
 )
 
 type removeOptions struct {
 	force     bool
 	noPrune   bool
-	platforms []string
+	platforms []ocispec.Platform
 }
 
 // NewRemoveCommand creates a new `docker remove` command
@@ -42,8 +43,7 @@ func NewRemoveCommand(dockerCLI command.Cli) *cobra.Command {
 	flags.BoolVarP(&options.force, "force", "f", false, "Force removal of the image")
 	flags.BoolVar(&options.noPrune, "no-prune", false, "Do not delete untagged parents")
 
-	// TODO(thaJeztah): create a "platforms" option for this (including validation / parsing).
-	flags.StringSliceVar(&options.platforms, "platform", nil, `Remove only the given platform variant. Formatted as "os[/arch[/variant]]" (e.g., "linux/amd64")`)
+	flags.Var(opts.NewPlatformSlice(&options.platforms), "platform", `Remove only the given platform variant. Formatted as "os[/arch[/variant]]" (e.g., "linux/amd64")`)
 	_ = flags.SetAnnotation("platform", "version", []string{"1.50"})
 
 	_ = cmd.RegisterFlagCompletionFunc("platform", completion.Platforms)
@@ -63,14 +63,7 @@ func runRemove(ctx context.Context, dockerCLI command.Cli, opts removeOptions, i
 	options := image.RemoveOptions{
 		Force:         opts.force,
 		PruneChildren: !opts.noPrune,
-	}
-
-	for _, v := range opts.platforms {
-		p, err := platforms.Parse(v)
-		if err != nil {
-			return err
-		}
-		options.Platforms = append(options.Platforms, p)
+		Platforms:     opts.platforms,
 	}
 
 	// TODO(thaJeztah): this logic can likely be simplified: do we want to print "not found" errors at all when using "force"?

docs/reference/commandline/image_rm.md

@@ -9,11 +9,11 @@ Remove one or more images
 
 ### Options
 
-| Name                      | Type          | Default | Description                                                                                      |
-|:--------------------------|:--------------|:--------|:-------------------------------------------------------------------------------------------------|
-| `-f`, `--force`           | `bool`        |         | Force removal of the image                                                                       |
-| `--no-prune`              | `bool`        |         | Do not delete untagged parents                                                                   |
-| [`--platform`](#platform) | `stringSlice` |         | Remove only the given platform variant. Formatted as `os[/arch[/variant]]` (e.g., `linux/amd64`) |
+| Name                      | Type        | Default | Description                                                                                      |
+|:--------------------------|:------------|:--------|:-------------------------------------------------------------------------------------------------|
+| `-f`, `--force`           | `bool`      |         | Force removal of the image                                                                       |
+| `--no-prune`              | `bool`      |         | Do not delete untagged parents                                                                   |
+| [`--platform`](#platform) | `platforms` |         | Remove only the given platform variant. Formatted as `os[/arch[/variant]]` (e.g., `linux/amd64`) |
 
 
 <!---MARKER_GEN_END-->

docs/reference/commandline/rmi.md

@@ -9,11 +9,11 @@ Remove one or more images
 
 ### Options
 
-| Name            | Type          | Default | Description                                                                                      |
-|:----------------|:--------------|:--------|:-------------------------------------------------------------------------------------------------|
-| `-f`, `--force` | `bool`        |         | Force removal of the image                                                                       |
-| `--no-prune`    | `bool`        |         | Do not delete untagged parents                                                                   |
-| `--platform`    | `stringSlice` |         | Remove only the given platform variant. Formatted as `os[/arch[/variant]]` (e.g., `linux/amd64`) |
+| Name            | Type        | Default | Description                                                                                      |
+|:----------------|:------------|:--------|:-------------------------------------------------------------------------------------------------|
+| `-f`, `--force` | `bool`      |         | Force removal of the image                                                                       |
+| `--no-prune`    | `bool`      |         | Do not delete untagged parents                                                                   |
+| `--platform`    | `platforms` |         | Remove only the given platform variant. Formatted as `os[/arch[/variant]]` (e.g., `linux/amd64`) |
 
 
 <!---MARKER_GEN_END-->