Prior to 90bc19b3ae, note_data.given was set alongside an -m, -C or -F parameter for add / append. Following 90bc19b3ae, note_data.given is only set if the notes data buffer length is non-zero, which results in GIT_EDITOR invocation if e.g. a zero length blob object is provided. Revert to pre-90bc19b3ae note_data.given behavior, to fix non-interactive callers. Fixes: 90bc19b3ae ("notes.c: introduce '--separator=<paragraph-break>' option") Link: https://github.com/ddiss/icyci/issues/12 Signed-off-by: David Disseldorp <ddiss@xxxxxxx> --- builtin/notes.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/builtin/notes.c b/builtin/notes.c index d9c356e354..3ccb3eb602 100644 --- a/builtin/notes.c +++ b/builtin/notes.c @@ -282,6 +282,7 @@ static int parse_msg_arg(const struct option *opt, const char *arg, int unset) ALLOC_GROW_BY(d->messages, d->msg_nr, 1, d->msg_alloc); d->messages[d->msg_nr - 1] = msg; msg->stripspace = STRIPSPACE; + d->given = 1; return 0; } @@ -302,6 +303,7 @@ static int parse_file_arg(const struct option *opt, const char *arg, int unset) ALLOC_GROW_BY(d->messages, d->msg_nr, 1, d->msg_alloc); d->messages[d->msg_nr - 1] = msg; msg->stripspace = STRIPSPACE; + d->given = 1; return 0; } @@ -335,6 +337,7 @@ static int parse_reuse_arg(const struct option *opt, const char *arg, int unset) ALLOC_GROW_BY(d->messages, d->msg_nr, 1, d->msg_alloc); d->messages[d->msg_nr - 1] = msg; msg->stripspace = NO_STRIPSPACE; + d->given = 1; return 0; } @@ -515,7 +518,6 @@ static int add(int argc, const char **argv, const char *prefix) if (d.msg_nr) concat_messages(&d); - d.given = !!d.buf.len; object_ref = argc > 1 ? argv[1] : "HEAD"; @@ -692,7 +694,6 @@ static int append_edit(int argc, const char **argv, const char *prefix) if (d.msg_nr) concat_messages(&d); - d.given = !!d.buf.len; if (d.given && edit) fprintf(stderr, _("The -m/-F/-c/-C options have been deprecated " -- 2.43.0