PersistentFlags support about cli HOT 6 CLOSED

This is basically reopening #692 since that was closed due to folks not understanding what persistent flags are. They are NOT about "saving a flag that was set in the previous invocation of the cli".

from cli.

Some might think you can just use common flags on the root command and sub commands to get this to work, but sadly it does not. The following test currently fails:

package test

import (
	"github.com/urfave/cli/v2"
	"testing"
)

func TestCommonFlags(t *testing.T) {

	commonFlags := []cli.Flag{
		&cli.StringFlag{
			Name: "result",
		},
	}

	result := ""
	app := &cli.App{
		Name:  "root",
		Flags: commonFlags,
		Commands: []*cli.Command{
			{
				Name:  "sub",
				Flags: commonFlags,
				Action: func(ctx *cli.Context) error {
					result = ctx.String("result")
					return nil
				},
			},
		},
	}

	if err := app.Run([]string{"root", "sub", "--result", "after"}); err != nil {
		t.Fatal(err)
	}
	if result != "after" {
		t.Fatalf("result is not after")
	}

	if err := app.Run([]string{"root", "--result", "before", "sub"}); err != nil {
		t.Fatal(err)
	}
	if result != "before" {
		t.Fatalf("result is not before")
	}

}

from cli.

@chirino Support for persistent flags was added in v3. Please try with that. You can mark a flag explicitly as persistent in which case it will be allowed in subcommands as well.

from cli.

@chirino I am closing this for now. If you see any problems with persistent flags in v3 we can create a new issue

from cli.

Thanks! I Tested v3 out and it's working nice.

from cli.

Well mostly working.. found this bug: #1826

from cli.