From 862fc69bff4a83efe72bf64fa26730698b058b87 Mon Sep 17 00:00:00 2001 From: Jarek Kowalski Date: Mon, 10 Feb 2020 20:33:19 -0800 Subject: [PATCH] cli: marked --ignore-{file,dir}-errors as enums, so that CLI parser enforces correct values --- cli/command_policy_set.go | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/cli/command_policy_set.go b/cli/command_policy_set.go index 69250cbfa..214bdd133 100644 --- a/cli/command_policy_set.go +++ b/cli/command_policy_set.go @@ -13,6 +13,8 @@ "github.com/kopia/kopia/snapshot/policy" ) +var booleanEnumValues = []string{"true", "false", "inherit"} + var ( policySetCommand = policyCommands.Command("set", "Set snapshot policy for a single directory, user@host or a global policy.") policySetTargets = policySetCommand.Arg("target", "Target of a policy ('global','user@host','@host') or a path").Strings() @@ -57,8 +59,8 @@ policySetMaxFileSize = policySetCommand.Flag("max-file-size", "Exclude files above given size").PlaceHolder("N").String() // Error handling behavior. - policyIgnoreFileErrors = policySetCommand.Flag("ignore-file-errors", "Ignore errors reading files while traversing ('true', 'false', 'inherit')").String() - policyIgnoreDirectoryErrors = policySetCommand.Flag("ignore-dir-errors", "Ignore errors reading directories while traversing ('true', 'false', 'inherit").String() + policyIgnoreFileErrors = policySetCommand.Flag("ignore-file-errors", "Ignore errors reading files while traversing ('true', 'false', 'inherit')").Enum(booleanEnumValues...) + policyIgnoreDirectoryErrors = policySetCommand.Flag("ignore-dir-errors", "Ignore errors reading directories while traversing ('true', 'false', 'inherit").Enum(booleanEnumValues...) // General policy. policySetInherit = policySetCommand.Flag(inheritPolicyString, "Enable or disable inheriting policies from the parent").BoolList()