opensource/podman
1
0
Fork 0
mirror of https://github.com/containers/podman.git synced 2025-06-23 02:18:13 +08:00
Code Issues Packages Projects Releases Wiki Activity

Remove unused archive flag from diff commands

Browse Source 
* Option left in images/diff.go CLI as comment implies it is needed for
  backwards compatibility.

```release-note
NONE
```

[NO NEW TESTS NEEDED]

Signed-off-by: Jhon Honce <jhonce@redhat.com>
This commit is contained in:
Jhon Honce
2022-05-25 08:18:46 -07:00
parent dbd4ee04b8
commit 338f012370
5 changed files with 7 additions and 17 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
cmd/podman/containers/diff.go
View File

@ -32,13 +32,9 @@ func init() {
Parent: containerCmd,
})
diffOpts = &entities.DiffOptions{}
diffOpts = new(entities.DiffOptions)
flags := diffCmd.Flags()
// FIXME: Why does this exists? It is not used anywhere.
flags.BoolVar(&diffOpts.Archive, "archive", true, "Save the diff as a tar archive")
_ = flags.MarkHidden("archive")
formatFlagName := "format"
flags.StringVar(&diffOpts.Format, formatFlagName, "", "Change the output format (json)")
_ = diffCmd.RegisterFlagCompletionFunc(formatFlagName, common.AutocompleteFormat(nil))

3
cmd/podman/diff.go
View File

@ -35,9 +35,6 @@ func init() {
Command: diffCmd,
})
flags := diffCmd.Flags()
// FIXME: Why does this exists? It is not used anywhere.
flags.BoolVar(&diffOpts.Archive, "archive", true, "Save the diff as a tar archive")
_ = flags.MarkHidden("archive")
formatFlagName := "format"
flags.StringVar(&diffOpts.Format, formatFlagName, "", "Change the output format (json)")

4
cmd/podman/diff/diff.go
View File

@ -13,7 +13,7 @@ import (
"github.com/spf13/cobra"
)
func Diff(cmd *cobra.Command, args []string, options entities.DiffOptions) error {
func Diff(_ *cobra.Command, args []string, options entities.DiffOptions) error {
results, err := registry.ContainerEngine().Diff(registry.GetContext(), args, options)
if err != nil {
return err
@ -63,7 +63,7 @@ func changesToTable(diffs *entities.DiffReport) error {
return nil
}
// IDOrLatestArgs used to validate a nameOrId was provided or the "--latest" flag
// ValidateContainerDiffArgs used to validate a nameOrId was provided or the "--latest" flag
func ValidateContainerDiffArgs(cmd *cobra.Command, args []string) error {
given, _ := cmd.Flags().GetBool("latest")
if len(args) > 0 && !given {

4
cmd/podman/images/diff.go
View File

@ -34,9 +34,7 @@ func init() {
}
func diffFlags(flags *pflag.FlagSet) {
diffOpts = &entities.DiffOptions{}
flags.BoolVar(&diffOpts.Archive, "archive", true, "Save the diff as a tar archive")
_ = flags.MarkDeprecated("archive", "Provided for backwards compatibility, has no impact on output.")
diffOpts = new(entities.DiffOptions)
formatFlagName := "format"
flags.StringVar(&diffOpts.Format, formatFlagName, "", "Change the output format (json)")

1
pkg/domain/entities/types.go
View File

@ -80,7 +80,6 @@ type InspectOptions struct {
type DiffOptions struct {
Format string `json:",omitempty"` // CLI only
Latest bool `json:",omitempty"` // API and CLI, only supported by containers
Archive bool `json:",omitempty"` // CLI only
Type define.DiffType // Type which should be compared
}