On Tue, Feb 26, 2019 at 4:54 PM Jacob Keller <jacob.e.keller@xxxxxxxxx> wrote:
> describe: bail of --contains --all is used with --exclude or --match
s/of/if/
> Signed-off-by: Jacob Keller <jacob.keller@xxxxxxxxx>
> ---
> diff --git a/builtin/describe.c b/builtin/describe.c
> @@ -589,6 +589,9 @@ int cmd_describe(int argc, const char **argv, const char *prefix)
> for_each_string_list_item(item, &exclude_patterns)
> argv_array_pushf(&args, "--exclude=refs/tags/%s", item->string);
> + } else {
> + if (patterns.nr || exclude_patterns.nr)
> + die(_("--contains with --all does not support --match or --exclude"));
> }
Could be folded into an 'else if':
} else if (patterns.nr || exclude_patterns.nr) {
die(...);
}
but not worth a re-roll.
