From: Paolo Bonzini <bonzini@gnu.org>
To: git@vger.kernel.org
Cc: Paolo Bonzini <pbonzini@redhat.com>
Subject: [PATCH 3/3] interpret-trailers: add options for actions
Date: Wed, 12 Jul 2017 15:46:46 +0200 [thread overview]
Message-ID: <20170712134646.17179-4-bonzini@gnu.org> (raw)
In-Reply-To: <20170712134646.17179-1-bonzini@gnu.org>
From: Paolo Bonzini <pbonzini@redhat.com>
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.
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
---
Documentation/git-interpret-trailers.txt | 16 ++++++++++
builtin/interpret-trailers.c | 40 +++++++++++++++++++++++++
t/t7513-interpret-trailers.sh | 51 ++++++++++++++++++++++++++++++++
trailer.c | 28 ++++++++++++++++--
trailer.h | 6 ++++
5 files changed, 138 insertions(+), 3 deletions(-)
diff --git a/Documentation/git-interpret-trailers.txt b/Documentation/git-interpret-trailers.txt
index 31cdeaecd..f4d67b8a1 100644
--- a/Documentation/git-interpret-trailers.txt
+++ b/Documentation/git-interpret-trailers.txt
@@ -80,6 +80,22 @@ OPTIONS
trailer to the input messages. See the description of this
command.
+--where <placement>::
+ Specify where all new trailers will be added. This option
+ overrides all configuration variables.
+
+--if-exists <action>::
+ Specify what action will be performed when there is already at
+ least one trailer with the same <token> in the message. This
+ option applies to all '--trailer' options and overrides all
+ configuration variables.
+
+--if-missing <action>::
+ Specify what action will be performed when there is no other
+ trailer with the same <token> in the message. This option
+ applies to all '--trailer' options and overrides all
+ configuration variables.
+
CONFIGURATION VARIABLES
-----------------------
diff --git a/builtin/interpret-trailers.c b/builtin/interpret-trailers.c
index 6528680b5..5f05f0af0 100644
--- a/builtin/interpret-trailers.c
+++ b/builtin/interpret-trailers.c
@@ -16,6 +16,39 @@ static const char * const git_interpret_trailers_usage[] = {
NULL
};
+static int option_parse_where(const struct option *opt,
+ const char *arg, int unset)
+{
+ enum action_where *where = opt->value;
+
+ if (unset)
+ return 0;
+
+ return set_where(where, arg);
+}
+
+static int option_parse_if_exists(const struct option *opt,
+ const char *arg, int unset)
+{
+ enum action_if_exists *if_exists = opt->value;
+
+ if (unset)
+ return 0;
+
+ return set_if_exists(if_exists, arg);
+}
+
+static int option_parse_if_missing(const struct option *opt,
+ const char *arg, int unset)
+{
+ enum action_if_missing *if_missing = opt->value;
+
+ if (unset)
+ return 0;
+
+ return set_if_missing(if_missing, arg);
+}
+
int cmd_interpret_trailers(int argc, const char **argv, const char *prefix)
{
struct trailer_opts opts = { 0 };
@@ -24,6 +57,13 @@ int cmd_interpret_trailers(int argc, const char **argv, const char *prefix)
struct option options[] = {
OPT_BOOL(0, "in-place", &opts.in_place, N_("edit files in place")),
OPT_BOOL(0, "trim-empty", &opts.trim_empty, N_("trim empty trailers")),
+ OPT_CALLBACK(0, "where", &opts.where, N_("action"),
+ N_("where to place the new trailer"), option_parse_where),
+ OPT_CALLBACK(0, "if-exists", &opts.if_exists, N_("action"),
+ N_("action if trailer already exists"), option_parse_if_exists),
+ OPT_CALLBACK(0, "if-missing", &opts.if_missing, N_("action"),
+ N_("action if trailer is missing"), option_parse_if_missing),
+
OPT_STRING_LIST(0, "trailer", &trailers, N_("trailer"),
N_("trailer(s) to add")),
OPT_END()
diff --git a/t/t7513-interpret-trailers.sh b/t/t7513-interpret-trailers.sh
index 0c6f91c43..49f4f823d 100755
--- a/t/t7513-interpret-trailers.sh
+++ b/t/t7513-interpret-trailers.sh
@@ -681,6 +681,21 @@ test_expect_success 'using "where = before"' '
test_cmp expected actual
'
+test_expect_success 'overriding configuration with "--where after"' '
+ git config trailer.ack.where "before" &&
+ cat complex_message_body >expected &&
+ sed -e "s/ Z\$/ /" >>expected <<-\EOF &&
+ Fixes: Z
+ Acked-by= Z
+ Acked-by= Peff
+ Reviewed-by: Z
+ Signed-off-by: Z
+ EOF
+ git interpret-trailers --where after --trailer "ack: Peff" \
+ complex_message >actual &&
+ test_cmp expected actual
+'
+
test_expect_success 'using "where = after"' '
git config trailer.ack.where "after" &&
cat complex_message_body >expected &&
@@ -947,6 +962,23 @@ test_expect_success 'using "ifExists = add" with "where = after"' '
test_cmp expected actual
'
+test_expect_success 'overriding configuration with "--if-exists replace"' '
+ git config trailer.fix.key "Fixes: " &&
+ git config trailer.fix.ifExists "add" &&
+ cat complex_message_body >expected &&
+ sed -e "s/ Z\$/ /" >>expected <<-\EOF &&
+ Bug #42
+ Acked-by= Z
+ Reviewed-by:
+ Signed-off-by: Z
+ Fixes: 22
+ EOF
+ git interpret-trailers --if-exists replace --trailer "review:" \
+ --trailer "fix=53" --trailer "fix=22" --trailer "bug: 42" \
+ <complex_message >actual &&
+ test_cmp expected actual
+'
+
test_expect_success 'using "ifExists = replace"' '
git config trailer.fix.key "Fixes: " &&
git config trailer.fix.ifExists "replace" &&
@@ -1026,6 +1058,25 @@ test_expect_success 'the default is "ifMissing = add"' '
test_cmp expected actual
'
+test_expect_success 'overriding configuration with "--if-missing doNothing"' '
+ git config trailer.ifmissing "add" &&
+ cat complex_message_body >expected &&
+ sed -e "s/ Z\$/ /" >>expected <<-\EOF &&
+ Fixes: Z
+ Acked-by= Z
+ Acked-by= Junio
+ Acked-by= Peff
+ Reviewed-by:
+ Signed-off-by: Z
+ EOF
+ git interpret-trailers --if-missing doNothing \
+ --trailer "review:" --trailer "fix=53" \
+ --trailer "cc=Linus" --trailer "ack: Junio" \
+ --trailer "fix=22" --trailer "bug: 42" --trailer "ack: Peff" \
+ <complex_message >actual &&
+ test_cmp expected actual
+'
+
test_expect_success 'when default "ifMissing" is "doNothing"' '
git config trailer.ifmissing "doNothing" &&
cat complex_message_body >expected &&
diff --git a/trailer.c b/trailer.c
index 558c52431..78e88f98d 100644
--- a/trailer.c
+++ b/trailer.c
@@ -549,14 +549,36 @@ static int git_trailer_config(const char *conf_key, const char *value, void *cb)
return 0;
}
-static void ensure_configured(void)
+static void apply_config_overrides(struct conf_info *conf, struct trailer_opts *opts)
{
+ if (!opts)
+ return;
+ if (opts->where != WHERE_DEFAULT)
+ conf->where = opts->where;
+ if (opts->if_exists != EXISTS_DEFAULT)
+ conf->if_exists = opts->if_exists;
+ if (opts->if_missing != MISSING_DEFAULT)
+ conf->if_missing = opts->if_missing;
+}
+
+static void ensure_configured(struct trailer_opts *opts)
+{
+ struct arg_item *item;
+ struct list_head *pos;
+
if (configured)
return;
/* Default config must be setup first */
git_config(git_trailer_default_config, NULL);
+ apply_config_overrides(&default_conf_info, opts);
+
git_config(git_trailer_config, NULL);
+ list_for_each(pos, &conf_head) {
+ item = list_entry(pos, struct arg_item, list);
+ apply_config_overrides(&item->conf, opts);
+ }
+
configured = 1;
}
@@ -977,7 +999,7 @@ void process_trailers(const char *file, struct trailer_opts *opts,
int trailer_end;
FILE *outfile = stdout;
- ensure_configured();
+ ensure_configured(opts);
read_input_file(&sb, file);
@@ -1013,7 +1035,7 @@ void trailer_info_get(struct trailer_info *info, const char *str)
size_t nr = 0, alloc = 0;
char **last = NULL;
- ensure_configured();
+ ensure_configured(NULL);
patch_start = find_patch_start(str);
trailer_end = find_trailer_end(str, patch_start);
diff --git a/trailer.h b/trailer.h
index f306bf059..634643c9d 100644
--- a/trailer.h
+++ b/trailer.h
@@ -2,12 +2,14 @@
#define TRAILER_H
enum action_where {
+ WHERE_DEFAULT,
WHERE_END,
WHERE_AFTER,
WHERE_BEFORE,
WHERE_START
};
enum action_if_exists {
+ EXISTS_DEFAULT,
EXISTS_ADD_IF_DIFFERENT_NEIGHBOR,
EXISTS_ADD_IF_DIFFERENT,
EXISTS_ADD,
@@ -15,6 +17,7 @@ enum action_if_exists {
EXISTS_DO_NOTHING
};
enum action_if_missing {
+ MISSING_DEFAULT,
MISSING_ADD,
MISSING_DO_NOTHING
};
@@ -22,6 +25,9 @@ enum action_if_missing {
struct trailer_opts {
int in_place;
int trim_empty;
+ enum action_where where;
+ enum action_if_exists if_exists;
+ enum action_if_missing if_missing;
};
int set_where(enum action_where *item, const char *value);
--
2.13.0
next prev parent reply other threads:[~2017-07-12 13:47 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-07-12 13:46 [PATCH 0/3] interpret-trailers: add --where, --if-exists, --if-missing Paolo Bonzini
2017-07-12 13:46 ` [PATCH 1/3] trailers: create struct trailer_opts Paolo Bonzini
2017-07-12 21:19 ` Jonathan Tan
2017-07-12 13:46 ` [PATCH 2/3] trailers: export action enums and corresponding lookup functions Paolo Bonzini
2017-07-12 21:13 ` Jonathan Tan
2017-07-12 13:46 ` Paolo Bonzini [this message]
2017-07-12 21:10 ` [PATCH 3/3] interpret-trailers: add options for actions Jonathan Tan
2017-07-12 21:33 ` Paolo Bonzini
2017-07-12 14:47 ` [PATCH 0/3] interpret-trailers: add --where, --if-exists, --if-missing Christian Couder
2017-07-12 15:02 ` Paolo Bonzini
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20170712134646.17179-4-bonzini@gnu.org \
--to=bonzini@gnu.org \
--cc=git@vger.kernel.org \
--cc=pbonzini@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).