mirror of
https://github.com/git/git.git
synced 2024-10-28 12:59:41 +01:00
Merge branch 'dd/notes-empty-no-edit-by-default' into maint-2.46
"git notes add -m '' --allow-empty" and friends that take prepared data to create notes should not invoke an editor, but it started doing so since Git 2.42, which has been corrected. * dd/notes-empty-no-edit-by-default: notes: do not trigger editor when adding an empty note
This commit is contained in:
commit
c09721cb63
2 changed files with 20 additions and 12 deletions
|
@ -114,7 +114,6 @@ struct note_msg {
|
|||
};
|
||||
|
||||
struct note_data {
|
||||
int given;
|
||||
int use_editor;
|
||||
int stripspace;
|
||||
char *edit_path;
|
||||
|
@ -193,7 +192,7 @@ static void write_commented_object(int fd, const struct object_id *object)
|
|||
static void prepare_note_data(const struct object_id *object, struct note_data *d,
|
||||
const struct object_id *old_note)
|
||||
{
|
||||
if (d->use_editor || !d->given) {
|
||||
if (d->use_editor || !d->msg_nr) {
|
||||
int fd;
|
||||
struct strbuf buf = STRBUF_INIT;
|
||||
|
||||
|
@ -201,7 +200,7 @@ static void prepare_note_data(const struct object_id *object, struct note_data *
|
|||
d->edit_path = git_pathdup("NOTES_EDITMSG");
|
||||
fd = xopen(d->edit_path, O_CREAT | O_TRUNC | O_WRONLY, 0600);
|
||||
|
||||
if (d->given)
|
||||
if (d->msg_nr)
|
||||
write_or_die(fd, d->buf.buf, d->buf.len);
|
||||
else if (old_note)
|
||||
copy_obj_to_fd(fd, old_note);
|
||||
|
@ -515,7 +514,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";
|
||||
|
||||
|
@ -528,7 +526,7 @@ static int add(int argc, const char **argv, const char *prefix)
|
|||
if (note) {
|
||||
if (!force) {
|
||||
free_notes(t);
|
||||
if (d.given) {
|
||||
if (d.msg_nr) {
|
||||
free_note_data(&d);
|
||||
return error(_("Cannot add notes. "
|
||||
"Found existing notes for object %s. "
|
||||
|
@ -690,14 +688,14 @@ static int append_edit(int argc, const char **argv, const char *prefix)
|
|||
usage_with_options(usage, options);
|
||||
}
|
||||
|
||||
if (d.msg_nr)
|
||||
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 "
|
||||
"for the 'edit' subcommand.\n"
|
||||
"Please use 'git notes add -f -m/-F/-c/-C' instead.\n"));
|
||||
if (edit)
|
||||
fprintf(stderr, _("The -m/-F/-c/-C options have been "
|
||||
"deprecated for the 'edit' subcommand.\n"
|
||||
"Please use 'git notes add -f -m/-F/-c/-C' "
|
||||
"instead.\n"));
|
||||
}
|
||||
|
||||
object_ref = 1 < argc ? argv[1] : "HEAD";
|
||||
|
||||
|
|
|
@ -1557,4 +1557,14 @@ test_expect_success 'empty notes are displayed by git log' '
|
|||
test_cmp expect actual
|
||||
'
|
||||
|
||||
test_expect_success 'empty notes do not invoke the editor' '
|
||||
test_commit 18th &&
|
||||
GIT_EDITOR="false" git notes add -C "$empty_blob" --allow-empty &&
|
||||
git notes remove HEAD &&
|
||||
GIT_EDITOR="false" git notes add -m "" --allow-empty &&
|
||||
git notes remove HEAD &&
|
||||
GIT_EDITOR="false" git notes add -F /dev/null --allow-empty &&
|
||||
git notes remove HEAD
|
||||
'
|
||||
|
||||
test_done
|
||||
|
|
Loading…
Reference in a new issue