diff options
author | Paolo Bonzini <pbonzini@redhat.com> | 2017-08-01 11:03:32 +0200 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2017-08-14 12:23:28 -0700 |
commit | 0ea5292e6be2f37b647b744177cfe6cc5e6f605d (patch) | |
tree | 6b2c409d6404c15ac21bf66db760291cd5210890 /builtin/interpret-trailers.c | |
parent | trailers: introduce struct new_trailer_item (diff) | |
download | tgif-0ea5292e6be2f37b647b744177cfe6cc5e6f605d.tar.xz |
interpret-trailers: add options for actions
Allow using non-default values for trailers without having to set
them up in .gitconfig first. For example, if you have the following
configuration
trailer.signed-off-by.where = end
you may use "--where before" when a patch author forgets his
Signed-off-by and provides it in a separate email. Likewise for
--if-exists and --if-missing
Reverting to the behavior specified by .gitconfig is done with
--no-where, --no-if-exists and --no-if-missing.
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin/interpret-trailers.c')
-rw-r--r-- | builtin/interpret-trailers.c | 32 |
1 files changed, 32 insertions, 0 deletions
diff --git a/builtin/interpret-trailers.c b/builtin/interpret-trailers.c index d12f36eda7..d7cbaf60f9 100644 --- a/builtin/interpret-trailers.c +++ b/builtin/interpret-trailers.c @@ -16,6 +16,28 @@ static const char * const git_interpret_trailers_usage[] = { NULL }; +static enum trailer_where where; +static enum trailer_if_exists if_exists; +static enum trailer_if_missing if_missing; + +static int option_parse_where(const struct option *opt, + const char *arg, int unset) +{ + return trailer_set_where(&where, arg); +} + +static int option_parse_if_exists(const struct option *opt, + const char *arg, int unset) +{ + return trailer_set_if_exists(&if_exists, arg); +} + +static int option_parse_if_missing(const struct option *opt, + const char *arg, int unset) +{ + return trailer_set_if_missing(&if_missing, arg); +} + static void new_trailers_clear(struct list_head *trailers) { struct list_head *pos, *tmp; @@ -44,6 +66,9 @@ static int option_parse_trailer(const struct option *opt, item = xmalloc(sizeof(*item)); item->text = arg; + item->where = where; + item->if_exists = if_exists; + item->if_missing = if_missing; list_add_tail(&item->list, trailers); return 0; } @@ -58,6 +83,13 @@ int cmd_interpret_trailers(int argc, const char **argv, const char *prefix) OPT_BOOL(0, "in-place", &in_place, N_("edit files in place")), OPT_BOOL(0, "trim-empty", &trim_empty, N_("trim empty trailers")), + OPT_CALLBACK(0, "where", NULL, N_("action"), + N_("where to place the new trailer"), option_parse_where), + OPT_CALLBACK(0, "if-exists", NULL, N_("action"), + N_("action if trailer already exists"), option_parse_if_exists), + OPT_CALLBACK(0, "if-missing", NULL, N_("action"), + N_("action if trailer is missing"), option_parse_if_missing), + OPT_CALLBACK(0, "trailer", &trailers, N_("trailer"), N_("trailer(s) to add"), option_parse_trailer), OPT_END() |