Johan Herland | 75ef3f4 | 2010-11-09 22:49:46 +0100 | [diff] [blame] | 1 | #include "cache.h" |
| 2 | #include "commit.h" |
| 3 | #include "refs.h" |
Stefan Beller | cbd53a2 | 2018-05-15 16:42:15 -0700 | [diff] [blame] | 4 | #include "object-store.h" |
Stefan Beller | 2122f67 | 2018-06-28 18:21:58 -0700 | [diff] [blame] | 5 | #include "repository.h" |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 6 | #include "diff.h" |
| 7 | #include "diffcore.h" |
Johan Herland | 809f38c | 2010-11-09 22:49:51 +0100 | [diff] [blame] | 8 | #include "xdiff-interface.h" |
| 9 | #include "ll-merge.h" |
| 10 | #include "dir.h" |
Johan Herland | 5688184 | 2010-11-09 22:49:47 +0100 | [diff] [blame] | 11 | #include "notes.h" |
Johan Herland | 75ef3f4 | 2010-11-09 22:49:46 +0100 | [diff] [blame] | 12 | #include "notes-merge.h" |
Johan Herland | 443259c | 2010-11-09 22:49:53 +0100 | [diff] [blame] | 13 | #include "strbuf.h" |
Johan Herland | bf9a05b | 2013-06-12 02:13:01 +0200 | [diff] [blame] | 14 | #include "notes-utils.h" |
Derrick Stolee | 6404355 | 2018-07-20 16:33:04 +0000 | [diff] [blame] | 15 | #include "commit-reach.h" |
Johan Herland | 75ef3f4 | 2010-11-09 22:49:46 +0100 | [diff] [blame] | 16 | |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 17 | struct notes_merge_pair { |
brian m. carlson | e910bb1 | 2016-09-05 20:08:01 +0000 | [diff] [blame] | 18 | struct object_id obj, base, local, remote; |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 19 | }; |
| 20 | |
Nguyễn Thái Ngọc Duy | 5684200 | 2018-11-10 06:48:53 +0100 | [diff] [blame] | 21 | void init_notes_merge_options(struct repository *r, |
| 22 | struct notes_merge_options *o) |
Johan Herland | 75ef3f4 | 2010-11-09 22:49:46 +0100 | [diff] [blame] | 23 | { |
| 24 | memset(o, 0, sizeof(struct notes_merge_options)); |
Johan Herland | 443259c | 2010-11-09 22:49:53 +0100 | [diff] [blame] | 25 | strbuf_init(&(o->commit_msg), 0); |
Johan Herland | 75ef3f4 | 2010-11-09 22:49:46 +0100 | [diff] [blame] | 26 | o->verbosity = NOTES_MERGE_VERBOSITY_DEFAULT; |
Nguyễn Thái Ngọc Duy | 5684200 | 2018-11-10 06:48:53 +0100 | [diff] [blame] | 27 | o->repo = r; |
Johan Herland | 75ef3f4 | 2010-11-09 22:49:46 +0100 | [diff] [blame] | 28 | } |
| 29 | |
Brandon Williams | 4d77896 | 2017-05-30 10:31:01 -0700 | [diff] [blame] | 30 | static int path_to_oid(const char *path, struct object_id *oid) |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 31 | { |
brian m. carlson | 2235030 | 2019-02-19 00:05:00 +0000 | [diff] [blame] | 32 | char hex_oid[GIT_MAX_HEXSZ]; |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 33 | int i = 0; |
brian m. carlson | 2235030 | 2019-02-19 00:05:00 +0000 | [diff] [blame] | 34 | while (*path && i < the_hash_algo->hexsz) { |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 35 | if (*path != '/') |
Brandon Williams | 4d77896 | 2017-05-30 10:31:01 -0700 | [diff] [blame] | 36 | hex_oid[i++] = *path; |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 37 | path++; |
| 38 | } |
brian m. carlson | 2235030 | 2019-02-19 00:05:00 +0000 | [diff] [blame] | 39 | if (*path || i != the_hash_algo->hexsz) |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 40 | return -1; |
Brandon Williams | 4d77896 | 2017-05-30 10:31:01 -0700 | [diff] [blame] | 41 | return get_oid_hex(hex_oid, oid); |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 42 | } |
| 43 | |
Brandon Williams | 4d77896 | 2017-05-30 10:31:01 -0700 | [diff] [blame] | 44 | static int verify_notes_filepair(struct diff_filepair *p, struct object_id *oid) |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 45 | { |
| 46 | switch (p->status) { |
| 47 | case DIFF_STATUS_MODIFIED: |
| 48 | assert(p->one->mode == p->two->mode); |
brian m. carlson | a0d12c4 | 2016-06-24 23:09:23 +0000 | [diff] [blame] | 49 | assert(!is_null_oid(&p->one->oid)); |
| 50 | assert(!is_null_oid(&p->two->oid)); |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 51 | break; |
| 52 | case DIFF_STATUS_ADDED: |
brian m. carlson | a0d12c4 | 2016-06-24 23:09:23 +0000 | [diff] [blame] | 53 | assert(is_null_oid(&p->one->oid)); |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 54 | break; |
| 55 | case DIFF_STATUS_DELETED: |
brian m. carlson | a0d12c4 | 2016-06-24 23:09:23 +0000 | [diff] [blame] | 56 | assert(is_null_oid(&p->two->oid)); |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 57 | break; |
| 58 | default: |
| 59 | return -1; |
| 60 | } |
| 61 | assert(!strcmp(p->one->path, p->two->path)); |
Brandon Williams | 4d77896 | 2017-05-30 10:31:01 -0700 | [diff] [blame] | 62 | return path_to_oid(p->one->path, oid); |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 63 | } |
| 64 | |
| 65 | static struct notes_merge_pair *find_notes_merge_pair_pos( |
Brandon Williams | d7a7c70 | 2017-05-30 10:31:00 -0700 | [diff] [blame] | 66 | struct notes_merge_pair *list, int len, struct object_id *obj, |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 67 | int insert_new, int *occupied) |
| 68 | { |
| 69 | /* |
| 70 | * Both diff_tree_remote() and diff_tree_local() tend to process |
| 71 | * merge_pairs in ascending order. Therefore, cache last returned |
| 72 | * index, and search sequentially from there until the appropriate |
| 73 | * position is found. |
| 74 | * |
| 75 | * Since inserts only happen from diff_tree_remote() (which mainly |
| 76 | * _appends_), we don't care that inserting into the middle of the |
| 77 | * list is expensive (using memmove()). |
| 78 | */ |
| 79 | static int last_index; |
| 80 | int i = last_index < len ? last_index : len - 1; |
| 81 | int prev_cmp = 0, cmp = -1; |
| 82 | while (i >= 0 && i < len) { |
Brandon Williams | d7a7c70 | 2017-05-30 10:31:00 -0700 | [diff] [blame] | 83 | cmp = oidcmp(obj, &list[i].obj); |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 84 | if (!cmp) /* obj belongs @ i */ |
| 85 | break; |
| 86 | else if (cmp < 0 && prev_cmp <= 0) /* obj belongs < i */ |
| 87 | i--; |
| 88 | else if (cmp < 0) /* obj belongs between i-1 and i */ |
| 89 | break; |
| 90 | else if (cmp > 0 && prev_cmp >= 0) /* obj belongs > i */ |
| 91 | i++; |
| 92 | else /* if (cmp > 0) */ { /* obj belongs between i and i+1 */ |
| 93 | i++; |
| 94 | break; |
| 95 | } |
| 96 | prev_cmp = cmp; |
| 97 | } |
| 98 | if (i < 0) |
| 99 | i = 0; |
| 100 | /* obj belongs at, or immediately preceding, index i (0 <= i <= len) */ |
| 101 | |
| 102 | if (!cmp) |
| 103 | *occupied = 1; |
| 104 | else { |
| 105 | *occupied = 0; |
| 106 | if (insert_new && i < len) { |
René Scharfe | f331ab9 | 2017-07-15 22:00:45 +0200 | [diff] [blame] | 107 | MOVE_ARRAY(list + i + 1, list + i, len - i); |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 108 | memset(list + i, 0, sizeof(struct notes_merge_pair)); |
| 109 | } |
| 110 | } |
| 111 | last_index = i; |
| 112 | return list + i; |
| 113 | } |
| 114 | |
brian m. carlson | e910bb1 | 2016-09-05 20:08:01 +0000 | [diff] [blame] | 115 | static struct object_id uninitialized = { |
Ævar Arnfjörð Bjarmason | 5010364 | 2022-03-17 18:27:17 +0100 | [diff] [blame] | 116 | .hash = |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 117 | "\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff" \ |
brian m. carlson | e910bb1 | 2016-09-05 20:08:01 +0000 | [diff] [blame] | 118 | "\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff" |
| 119 | }; |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 120 | |
| 121 | static struct notes_merge_pair *diff_tree_remote(struct notes_merge_options *o, |
Brandon Williams | 9d6babb | 2017-05-30 10:30:59 -0700 | [diff] [blame] | 122 | const struct object_id *base, |
| 123 | const struct object_id *remote, |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 124 | int *num_changes) |
| 125 | { |
| 126 | struct diff_options opt; |
| 127 | struct notes_merge_pair *changes; |
| 128 | int i, len = 0; |
| 129 | |
| 130 | trace_printf("\tdiff_tree_remote(base = %.7s, remote = %.7s)\n", |
Brandon Williams | 9d6babb | 2017-05-30 10:30:59 -0700 | [diff] [blame] | 131 | oid_to_hex(base), oid_to_hex(remote)); |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 132 | |
Nguyễn Thái Ngọc Duy | 5684200 | 2018-11-10 06:48:53 +0100 | [diff] [blame] | 133 | repo_diff_setup(o->repo, &opt); |
Brandon Williams | 0d1e0e7 | 2017-10-31 11:19:11 -0700 | [diff] [blame] | 134 | opt.flags.recursive = 1; |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 135 | opt.output_format = DIFF_FORMAT_NO_OUTPUT; |
Thomas Rast | 2845265 | 2012-08-03 14:16:24 +0200 | [diff] [blame] | 136 | diff_setup_done(&opt); |
Brandon Williams | 66f414f | 2017-05-30 10:31:03 -0700 | [diff] [blame] | 137 | diff_tree_oid(base, remote, "", &opt); |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 138 | diffcore_std(&opt); |
| 139 | |
René Scharfe | ca56dad | 2021-03-13 17:17:22 +0100 | [diff] [blame] | 140 | CALLOC_ARRAY(changes, diff_queued_diff.nr); |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 141 | |
| 142 | for (i = 0; i < diff_queued_diff.nr; i++) { |
| 143 | struct diff_filepair *p = diff_queued_diff.queue[i]; |
| 144 | struct notes_merge_pair *mp; |
| 145 | int occupied; |
Brandon Williams | d7a7c70 | 2017-05-30 10:31:00 -0700 | [diff] [blame] | 146 | struct object_id obj; |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 147 | |
Brandon Williams | 4d77896 | 2017-05-30 10:31:01 -0700 | [diff] [blame] | 148 | if (verify_notes_filepair(p, &obj)) { |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 149 | trace_printf("\t\tCannot merge entry '%s' (%c): " |
| 150 | "%.7s -> %.7s. Skipping!\n", p->one->path, |
brian m. carlson | a0d12c4 | 2016-06-24 23:09:23 +0000 | [diff] [blame] | 151 | p->status, oid_to_hex(&p->one->oid), |
| 152 | oid_to_hex(&p->two->oid)); |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 153 | continue; |
| 154 | } |
Brandon Williams | d7a7c70 | 2017-05-30 10:31:00 -0700 | [diff] [blame] | 155 | mp = find_notes_merge_pair_pos(changes, len, &obj, 1, &occupied); |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 156 | if (occupied) { |
| 157 | /* We've found an addition/deletion pair */ |
Jeff King | 4a7e27e | 2018-08-28 17:22:40 -0400 | [diff] [blame] | 158 | assert(oideq(&mp->obj, &obj)); |
brian m. carlson | a0d12c4 | 2016-06-24 23:09:23 +0000 | [diff] [blame] | 159 | if (is_null_oid(&p->one->oid)) { /* addition */ |
brian m. carlson | e910bb1 | 2016-09-05 20:08:01 +0000 | [diff] [blame] | 160 | assert(is_null_oid(&mp->remote)); |
| 161 | oidcpy(&mp->remote, &p->two->oid); |
brian m. carlson | a0d12c4 | 2016-06-24 23:09:23 +0000 | [diff] [blame] | 162 | } else if (is_null_oid(&p->two->oid)) { /* deletion */ |
brian m. carlson | e910bb1 | 2016-09-05 20:08:01 +0000 | [diff] [blame] | 163 | assert(is_null_oid(&mp->base)); |
| 164 | oidcpy(&mp->base, &p->one->oid); |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 165 | } else |
| 166 | assert(!"Invalid existing change recorded"); |
| 167 | } else { |
Brandon Williams | d7a7c70 | 2017-05-30 10:31:00 -0700 | [diff] [blame] | 168 | oidcpy(&mp->obj, &obj); |
brian m. carlson | e910bb1 | 2016-09-05 20:08:01 +0000 | [diff] [blame] | 169 | oidcpy(&mp->base, &p->one->oid); |
| 170 | oidcpy(&mp->local, &uninitialized); |
| 171 | oidcpy(&mp->remote, &p->two->oid); |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 172 | len++; |
| 173 | } |
| 174 | trace_printf("\t\tStored remote change for %s: %.7s -> %.7s\n", |
brian m. carlson | e910bb1 | 2016-09-05 20:08:01 +0000 | [diff] [blame] | 175 | oid_to_hex(&mp->obj), oid_to_hex(&mp->base), |
| 176 | oid_to_hex(&mp->remote)); |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 177 | } |
| 178 | diff_flush(&opt); |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 179 | |
| 180 | *num_changes = len; |
| 181 | return changes; |
| 182 | } |
| 183 | |
| 184 | static void diff_tree_local(struct notes_merge_options *o, |
| 185 | struct notes_merge_pair *changes, int len, |
Brandon Williams | 9d6babb | 2017-05-30 10:30:59 -0700 | [diff] [blame] | 186 | const struct object_id *base, |
| 187 | const struct object_id *local) |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 188 | { |
| 189 | struct diff_options opt; |
| 190 | int i; |
| 191 | |
| 192 | trace_printf("\tdiff_tree_local(len = %i, base = %.7s, local = %.7s)\n", |
Brandon Williams | 9d6babb | 2017-05-30 10:30:59 -0700 | [diff] [blame] | 193 | len, oid_to_hex(base), oid_to_hex(local)); |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 194 | |
Nguyễn Thái Ngọc Duy | 5684200 | 2018-11-10 06:48:53 +0100 | [diff] [blame] | 195 | repo_diff_setup(o->repo, &opt); |
Brandon Williams | 0d1e0e7 | 2017-10-31 11:19:11 -0700 | [diff] [blame] | 196 | opt.flags.recursive = 1; |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 197 | opt.output_format = DIFF_FORMAT_NO_OUTPUT; |
Thomas Rast | 2845265 | 2012-08-03 14:16:24 +0200 | [diff] [blame] | 198 | diff_setup_done(&opt); |
Brandon Williams | 66f414f | 2017-05-30 10:31:03 -0700 | [diff] [blame] | 199 | diff_tree_oid(base, local, "", &opt); |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 200 | diffcore_std(&opt); |
| 201 | |
| 202 | for (i = 0; i < diff_queued_diff.nr; i++) { |
| 203 | struct diff_filepair *p = diff_queued_diff.queue[i]; |
| 204 | struct notes_merge_pair *mp; |
| 205 | int match; |
Brandon Williams | d7a7c70 | 2017-05-30 10:31:00 -0700 | [diff] [blame] | 206 | struct object_id obj; |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 207 | |
Brandon Williams | 4d77896 | 2017-05-30 10:31:01 -0700 | [diff] [blame] | 208 | if (verify_notes_filepair(p, &obj)) { |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 209 | trace_printf("\t\tCannot merge entry '%s' (%c): " |
| 210 | "%.7s -> %.7s. Skipping!\n", p->one->path, |
brian m. carlson | a0d12c4 | 2016-06-24 23:09:23 +0000 | [diff] [blame] | 211 | p->status, oid_to_hex(&p->one->oid), |
| 212 | oid_to_hex(&p->two->oid)); |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 213 | continue; |
| 214 | } |
Brandon Williams | d7a7c70 | 2017-05-30 10:31:00 -0700 | [diff] [blame] | 215 | mp = find_notes_merge_pair_pos(changes, len, &obj, 0, &match); |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 216 | if (!match) { |
| 217 | trace_printf("\t\tIgnoring local-only change for %s: " |
Brandon Williams | d7a7c70 | 2017-05-30 10:31:00 -0700 | [diff] [blame] | 218 | "%.7s -> %.7s\n", oid_to_hex(&obj), |
brian m. carlson | a0d12c4 | 2016-06-24 23:09:23 +0000 | [diff] [blame] | 219 | oid_to_hex(&p->one->oid), |
| 220 | oid_to_hex(&p->two->oid)); |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 221 | continue; |
| 222 | } |
| 223 | |
Jeff King | 4a7e27e | 2018-08-28 17:22:40 -0400 | [diff] [blame] | 224 | assert(oideq(&mp->obj, &obj)); |
brian m. carlson | a0d12c4 | 2016-06-24 23:09:23 +0000 | [diff] [blame] | 225 | if (is_null_oid(&p->two->oid)) { /* deletion */ |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 226 | /* |
| 227 | * Either this is a true deletion (1), or it is part |
| 228 | * of an A/D pair (2), or D/A pair (3): |
| 229 | * |
| 230 | * (1) mp->local is uninitialized; set it to null_sha1 |
| 231 | * (2) mp->local is not uninitialized; don't touch it |
| 232 | * (3) mp->local is uninitialized; set it to null_sha1 |
| 233 | * (will be overwritten by following addition) |
| 234 | */ |
Jeff King | 4a7e27e | 2018-08-28 17:22:40 -0400 | [diff] [blame] | 235 | if (oideq(&mp->local, &uninitialized)) |
brian m. carlson | e910bb1 | 2016-09-05 20:08:01 +0000 | [diff] [blame] | 236 | oidclr(&mp->local); |
brian m. carlson | a0d12c4 | 2016-06-24 23:09:23 +0000 | [diff] [blame] | 237 | } else if (is_null_oid(&p->one->oid)) { /* addition */ |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 238 | /* |
| 239 | * Either this is a true addition (1), or it is part |
| 240 | * of an A/D pair (2), or D/A pair (3): |
| 241 | * |
| 242 | * (1) mp->local is uninitialized; set to p->two->sha1 |
| 243 | * (2) mp->local is uninitialized; set to p->two->sha1 |
| 244 | * (3) mp->local is null_sha1; set to p->two->sha1 |
| 245 | */ |
brian m. carlson | e910bb1 | 2016-09-05 20:08:01 +0000 | [diff] [blame] | 246 | assert(is_null_oid(&mp->local) || |
Jeff King | 4a7e27e | 2018-08-28 17:22:40 -0400 | [diff] [blame] | 247 | oideq(&mp->local, &uninitialized)); |
brian m. carlson | e910bb1 | 2016-09-05 20:08:01 +0000 | [diff] [blame] | 248 | oidcpy(&mp->local, &p->two->oid); |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 249 | } else { /* modification */ |
| 250 | /* |
| 251 | * This is a true modification. p->one->sha1 shall |
| 252 | * match mp->base, and mp->local shall be uninitialized. |
| 253 | * Set mp->local to p->two->sha1. |
| 254 | */ |
Jeff King | 4a7e27e | 2018-08-28 17:22:40 -0400 | [diff] [blame] | 255 | assert(oideq(&p->one->oid, &mp->base)); |
| 256 | assert(oideq(&mp->local, &uninitialized)); |
brian m. carlson | e910bb1 | 2016-09-05 20:08:01 +0000 | [diff] [blame] | 257 | oidcpy(&mp->local, &p->two->oid); |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 258 | } |
| 259 | trace_printf("\t\tStored local change for %s: %.7s -> %.7s\n", |
brian m. carlson | e910bb1 | 2016-09-05 20:08:01 +0000 | [diff] [blame] | 260 | oid_to_hex(&mp->obj), oid_to_hex(&mp->base), |
| 261 | oid_to_hex(&mp->local)); |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 262 | } |
| 263 | diff_flush(&opt); |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 264 | } |
| 265 | |
Johan Herland | 809f38c | 2010-11-09 22:49:51 +0100 | [diff] [blame] | 266 | static void check_notes_merge_worktree(struct notes_merge_options *o) |
| 267 | { |
| 268 | if (!o->has_worktree) { |
| 269 | /* |
| 270 | * Must establish NOTES_MERGE_WORKTREE. |
| 271 | * Abort if NOTES_MERGE_WORKTREE already exists |
| 272 | */ |
Johan Herland | dabba59 | 2012-03-15 15:58:56 +0100 | [diff] [blame] | 273 | if (file_exists(git_path(NOTES_MERGE_WORKTREE)) && |
| 274 | !is_empty_dir(git_path(NOTES_MERGE_WORKTREE))) { |
Ben Boeckel | ed9bff0 | 2021-08-23 12:44:00 +0200 | [diff] [blame] | 275 | if (advice_enabled(ADVICE_RESOLVE_CONFLICT)) |
Vasco Almeida | c041c6d | 2016-09-19 13:08:20 +0000 | [diff] [blame] | 276 | die(_("You have not concluded your previous " |
Johan Herland | 809f38c | 2010-11-09 22:49:51 +0100 | [diff] [blame] | 277 | "notes merge (%s exists).\nPlease, use " |
| 278 | "'git notes merge --commit' or 'git notes " |
Johan Herland | 6abb365 | 2010-11-09 22:49:52 +0100 | [diff] [blame] | 279 | "merge --abort' to commit/abort the " |
Johan Herland | 809f38c | 2010-11-09 22:49:51 +0100 | [diff] [blame] | 280 | "previous merge before you start a new " |
Vasco Almeida | c041c6d | 2016-09-19 13:08:20 +0000 | [diff] [blame] | 281 | "notes merge."), git_path("NOTES_MERGE_*")); |
Johan Herland | 809f38c | 2010-11-09 22:49:51 +0100 | [diff] [blame] | 282 | else |
Vasco Almeida | c041c6d | 2016-09-19 13:08:20 +0000 | [diff] [blame] | 283 | die(_("You have not concluded your notes merge " |
| 284 | "(%s exists)."), git_path("NOTES_MERGE_*")); |
Johan Herland | 809f38c | 2010-11-09 22:49:51 +0100 | [diff] [blame] | 285 | } |
| 286 | |
Nguyễn Thái Ngọc Duy | dcf6926 | 2014-11-30 15:24:27 +0700 | [diff] [blame] | 287 | if (safe_create_leading_directories_const(git_path( |
Johan Herland | 809f38c | 2010-11-09 22:49:51 +0100 | [diff] [blame] | 288 | NOTES_MERGE_WORKTREE "/.test"))) |
| 289 | die_errno("unable to create directory %s", |
| 290 | git_path(NOTES_MERGE_WORKTREE)); |
| 291 | o->has_worktree = 1; |
| 292 | } else if (!file_exists(git_path(NOTES_MERGE_WORKTREE))) |
| 293 | /* NOTES_MERGE_WORKTREE should already be established */ |
| 294 | die("missing '%s'. This should not happen", |
| 295 | git_path(NOTES_MERGE_WORKTREE)); |
| 296 | } |
| 297 | |
Brandon Williams | 9e5e0c2 | 2017-05-30 10:31:02 -0700 | [diff] [blame] | 298 | static void write_buf_to_worktree(const struct object_id *obj, |
Johan Herland | 809f38c | 2010-11-09 22:49:51 +0100 | [diff] [blame] | 299 | const char *buf, unsigned long size) |
| 300 | { |
| 301 | int fd; |
Brandon Williams | 9e5e0c2 | 2017-05-30 10:31:02 -0700 | [diff] [blame] | 302 | char *path = git_pathdup(NOTES_MERGE_WORKTREE "/%s", oid_to_hex(obj)); |
Nguyễn Thái Ngọc Duy | dcf6926 | 2014-11-30 15:24:27 +0700 | [diff] [blame] | 303 | if (safe_create_leading_directories_const(path)) |
Johan Herland | 809f38c | 2010-11-09 22:49:51 +0100 | [diff] [blame] | 304 | die_errno("unable to create directory for '%s'", path); |
Johan Herland | 809f38c | 2010-11-09 22:49:51 +0100 | [diff] [blame] | 305 | |
René Scharfe | deb9c15 | 2016-07-07 22:08:30 +0200 | [diff] [blame] | 306 | fd = xopen(path, O_WRONLY | O_EXCL | O_CREAT, 0666); |
Johan Herland | 809f38c | 2010-11-09 22:49:51 +0100 | [diff] [blame] | 307 | |
| 308 | while (size > 0) { |
Jeff King | 634eb82 | 2017-09-13 13:17:44 -0400 | [diff] [blame] | 309 | ssize_t ret = write_in_full(fd, buf, size); |
Johan Herland | 809f38c | 2010-11-09 22:49:51 +0100 | [diff] [blame] | 310 | if (ret < 0) { |
| 311 | /* Ignore epipe */ |
| 312 | if (errno == EPIPE) |
| 313 | break; |
| 314 | die_errno("notes-merge"); |
Johan Herland | 809f38c | 2010-11-09 22:49:51 +0100 | [diff] [blame] | 315 | } |
| 316 | size -= ret; |
| 317 | buf += ret; |
| 318 | } |
| 319 | |
| 320 | close(fd); |
Jeff King | fcd12db | 2015-08-10 05:35:31 -0400 | [diff] [blame] | 321 | free(path); |
Johan Herland | 809f38c | 2010-11-09 22:49:51 +0100 | [diff] [blame] | 322 | } |
| 323 | |
Brandon Williams | 9e5e0c2 | 2017-05-30 10:31:02 -0700 | [diff] [blame] | 324 | static void write_note_to_worktree(const struct object_id *obj, |
| 325 | const struct object_id *note) |
Johan Herland | 809f38c | 2010-11-09 22:49:51 +0100 | [diff] [blame] | 326 | { |
| 327 | enum object_type type; |
| 328 | unsigned long size; |
brian m. carlson | b4f5aca | 2018-03-12 02:27:53 +0000 | [diff] [blame] | 329 | void *buf = read_object_file(note, &type, &size); |
Johan Herland | 809f38c | 2010-11-09 22:49:51 +0100 | [diff] [blame] | 330 | |
| 331 | if (!buf) |
| 332 | die("cannot read note %s for object %s", |
Brandon Williams | 9e5e0c2 | 2017-05-30 10:31:02 -0700 | [diff] [blame] | 333 | oid_to_hex(note), oid_to_hex(obj)); |
Johan Herland | 809f38c | 2010-11-09 22:49:51 +0100 | [diff] [blame] | 334 | if (type != OBJ_BLOB) |
| 335 | die("blob expected in note %s for object %s", |
Brandon Williams | 9e5e0c2 | 2017-05-30 10:31:02 -0700 | [diff] [blame] | 336 | oid_to_hex(note), oid_to_hex(obj)); |
Johan Herland | 809f38c | 2010-11-09 22:49:51 +0100 | [diff] [blame] | 337 | write_buf_to_worktree(obj, buf, size); |
| 338 | free(buf); |
| 339 | } |
| 340 | |
| 341 | static int ll_merge_in_worktree(struct notes_merge_options *o, |
| 342 | struct notes_merge_pair *p) |
| 343 | { |
| 344 | mmbuffer_t result_buf; |
| 345 | mmfile_t base, local, remote; |
Elijah Newren | 35f6967 | 2022-02-02 02:37:30 +0000 | [diff] [blame] | 346 | enum ll_merge_result status; |
Johan Herland | 809f38c | 2010-11-09 22:49:51 +0100 | [diff] [blame] | 347 | |
brian m. carlson | d449347 | 2016-09-05 20:08:02 +0000 | [diff] [blame] | 348 | read_mmblob(&base, &p->base); |
| 349 | read_mmblob(&local, &p->local); |
| 350 | read_mmblob(&remote, &p->remote); |
Johan Herland | 809f38c | 2010-11-09 22:49:51 +0100 | [diff] [blame] | 351 | |
brian m. carlson | e910bb1 | 2016-09-05 20:08:01 +0000 | [diff] [blame] | 352 | status = ll_merge(&result_buf, oid_to_hex(&p->obj), &base, NULL, |
Nguyễn Thái Ngọc Duy | 32eaa46 | 2018-09-21 17:57:27 +0200 | [diff] [blame] | 353 | &local, o->local_ref, &remote, o->remote_ref, |
Nguyễn Thái Ngọc Duy | 5684200 | 2018-11-10 06:48:53 +0100 | [diff] [blame] | 354 | o->repo->index, NULL); |
Johan Herland | 809f38c | 2010-11-09 22:49:51 +0100 | [diff] [blame] | 355 | |
| 356 | free(base.ptr); |
| 357 | free(local.ptr); |
| 358 | free(remote.ptr); |
| 359 | |
Elijah Newren | 35f6967 | 2022-02-02 02:37:30 +0000 | [diff] [blame] | 360 | if (status == LL_MERGE_BINARY_CONFLICT) |
| 361 | warning("Cannot merge binary files: %s (%s vs. %s)", |
| 362 | oid_to_hex(&p->obj), o->local_ref, o->remote_ref); |
Johan Herland | 809f38c | 2010-11-09 22:49:51 +0100 | [diff] [blame] | 363 | if ((status < 0) || !result_buf.ptr) |
| 364 | die("Failed to execute internal merge"); |
| 365 | |
Brandon Williams | 9e5e0c2 | 2017-05-30 10:31:02 -0700 | [diff] [blame] | 366 | write_buf_to_worktree(&p->obj, result_buf.ptr, result_buf.size); |
Johan Herland | 809f38c | 2010-11-09 22:49:51 +0100 | [diff] [blame] | 367 | free(result_buf.ptr); |
| 368 | |
| 369 | return status; |
| 370 | } |
| 371 | |
| 372 | static int merge_one_change_manual(struct notes_merge_options *o, |
| 373 | struct notes_merge_pair *p, |
| 374 | struct notes_tree *t) |
| 375 | { |
| 376 | const char *lref = o->local_ref ? o->local_ref : "local version"; |
| 377 | const char *rref = o->remote_ref ? o->remote_ref : "remote version"; |
| 378 | |
| 379 | trace_printf("\t\t\tmerge_one_change_manual(obj = %.7s, base = %.7s, " |
| 380 | "local = %.7s, remote = %.7s)\n", |
brian m. carlson | e910bb1 | 2016-09-05 20:08:01 +0000 | [diff] [blame] | 381 | oid_to_hex(&p->obj), oid_to_hex(&p->base), |
| 382 | oid_to_hex(&p->local), oid_to_hex(&p->remote)); |
Johan Herland | 809f38c | 2010-11-09 22:49:51 +0100 | [diff] [blame] | 383 | |
Johan Herland | 443259c | 2010-11-09 22:49:53 +0100 | [diff] [blame] | 384 | /* add "Conflicts:" section to commit message first time through */ |
| 385 | if (!o->has_worktree) |
| 386 | strbuf_addstr(&(o->commit_msg), "\n\nConflicts:\n"); |
| 387 | |
brian m. carlson | e910bb1 | 2016-09-05 20:08:01 +0000 | [diff] [blame] | 388 | strbuf_addf(&(o->commit_msg), "\t%s\n", oid_to_hex(&p->obj)); |
Johan Herland | 443259c | 2010-11-09 22:49:53 +0100 | [diff] [blame] | 389 | |
Jonathan Nieder | 5f9f8d1 | 2011-11-17 19:27:46 -0600 | [diff] [blame] | 390 | if (o->verbosity >= 2) |
brian m. carlson | e910bb1 | 2016-09-05 20:08:01 +0000 | [diff] [blame] | 391 | printf("Auto-merging notes for %s\n", oid_to_hex(&p->obj)); |
Johan Herland | 809f38c | 2010-11-09 22:49:51 +0100 | [diff] [blame] | 392 | check_notes_merge_worktree(o); |
brian m. carlson | e910bb1 | 2016-09-05 20:08:01 +0000 | [diff] [blame] | 393 | if (is_null_oid(&p->local)) { |
Johan Herland | 809f38c | 2010-11-09 22:49:51 +0100 | [diff] [blame] | 394 | /* D/F conflict, checkout p->remote */ |
brian m. carlson | e910bb1 | 2016-09-05 20:08:01 +0000 | [diff] [blame] | 395 | assert(!is_null_oid(&p->remote)); |
Jonathan Nieder | 5f9f8d1 | 2011-11-17 19:27:46 -0600 | [diff] [blame] | 396 | if (o->verbosity >= 1) |
| 397 | printf("CONFLICT (delete/modify): Notes for object %s " |
| 398 | "deleted in %s and modified in %s. Version from %s " |
| 399 | "left in tree.\n", |
brian m. carlson | e910bb1 | 2016-09-05 20:08:01 +0000 | [diff] [blame] | 400 | oid_to_hex(&p->obj), lref, rref, rref); |
Brandon Williams | 9e5e0c2 | 2017-05-30 10:31:02 -0700 | [diff] [blame] | 401 | write_note_to_worktree(&p->obj, &p->remote); |
brian m. carlson | e910bb1 | 2016-09-05 20:08:01 +0000 | [diff] [blame] | 402 | } else if (is_null_oid(&p->remote)) { |
Johan Herland | 809f38c | 2010-11-09 22:49:51 +0100 | [diff] [blame] | 403 | /* D/F conflict, checkout p->local */ |
brian m. carlson | e910bb1 | 2016-09-05 20:08:01 +0000 | [diff] [blame] | 404 | assert(!is_null_oid(&p->local)); |
Jonathan Nieder | 5f9f8d1 | 2011-11-17 19:27:46 -0600 | [diff] [blame] | 405 | if (o->verbosity >= 1) |
| 406 | printf("CONFLICT (delete/modify): Notes for object %s " |
| 407 | "deleted in %s and modified in %s. Version from %s " |
| 408 | "left in tree.\n", |
brian m. carlson | e910bb1 | 2016-09-05 20:08:01 +0000 | [diff] [blame] | 409 | oid_to_hex(&p->obj), rref, lref, lref); |
Brandon Williams | 9e5e0c2 | 2017-05-30 10:31:02 -0700 | [diff] [blame] | 410 | write_note_to_worktree(&p->obj, &p->local); |
Johan Herland | 809f38c | 2010-11-09 22:49:51 +0100 | [diff] [blame] | 411 | } else { |
| 412 | /* "regular" conflict, checkout result of ll_merge() */ |
| 413 | const char *reason = "content"; |
brian m. carlson | e910bb1 | 2016-09-05 20:08:01 +0000 | [diff] [blame] | 414 | if (is_null_oid(&p->base)) |
Johan Herland | 809f38c | 2010-11-09 22:49:51 +0100 | [diff] [blame] | 415 | reason = "add/add"; |
brian m. carlson | e910bb1 | 2016-09-05 20:08:01 +0000 | [diff] [blame] | 416 | assert(!is_null_oid(&p->local)); |
| 417 | assert(!is_null_oid(&p->remote)); |
Jonathan Nieder | 5f9f8d1 | 2011-11-17 19:27:46 -0600 | [diff] [blame] | 418 | if (o->verbosity >= 1) |
| 419 | printf("CONFLICT (%s): Merge conflict in notes for " |
brian m. carlson | e910bb1 | 2016-09-05 20:08:01 +0000 | [diff] [blame] | 420 | "object %s\n", reason, |
| 421 | oid_to_hex(&p->obj)); |
Johan Herland | 809f38c | 2010-11-09 22:49:51 +0100 | [diff] [blame] | 422 | ll_merge_in_worktree(o, p); |
| 423 | } |
| 424 | |
| 425 | trace_printf("\t\t\tremoving from partial merge result\n"); |
brian m. carlson | e910bb1 | 2016-09-05 20:08:01 +0000 | [diff] [blame] | 426 | remove_note(t, p->obj.hash); |
Johan Herland | 809f38c | 2010-11-09 22:49:51 +0100 | [diff] [blame] | 427 | |
| 428 | return 1; |
| 429 | } |
| 430 | |
Johan Herland | 3228e67 | 2010-11-15 00:55:12 +0100 | [diff] [blame] | 431 | static int merge_one_change(struct notes_merge_options *o, |
| 432 | struct notes_merge_pair *p, struct notes_tree *t) |
| 433 | { |
| 434 | /* |
Johan Herland | 809f38c | 2010-11-09 22:49:51 +0100 | [diff] [blame] | 435 | * Return 0 if change is successfully resolved (stored in notes_tree). |
| 436 | * Return 1 is change results in a conflict (NOT stored in notes_tree, |
| 437 | * but instead written to NOTES_MERGE_WORKTREE with conflict markers). |
Johan Herland | 3228e67 | 2010-11-15 00:55:12 +0100 | [diff] [blame] | 438 | */ |
| 439 | switch (o->strategy) { |
| 440 | case NOTES_MERGE_RESOLVE_MANUAL: |
Johan Herland | 809f38c | 2010-11-09 22:49:51 +0100 | [diff] [blame] | 441 | return merge_one_change_manual(o, p, t); |
Johan Herland | 3228e67 | 2010-11-15 00:55:12 +0100 | [diff] [blame] | 442 | case NOTES_MERGE_RESOLVE_OURS: |
Jonathan Nieder | 5f9f8d1 | 2011-11-17 19:27:46 -0600 | [diff] [blame] | 443 | if (o->verbosity >= 2) |
| 444 | printf("Using local notes for %s\n", |
brian m. carlson | e910bb1 | 2016-09-05 20:08:01 +0000 | [diff] [blame] | 445 | oid_to_hex(&p->obj)); |
Johan Herland | 3228e67 | 2010-11-15 00:55:12 +0100 | [diff] [blame] | 446 | /* nothing to do */ |
| 447 | return 0; |
| 448 | case NOTES_MERGE_RESOLVE_THEIRS: |
Jonathan Nieder | 5f9f8d1 | 2011-11-17 19:27:46 -0600 | [diff] [blame] | 449 | if (o->verbosity >= 2) |
| 450 | printf("Using remote notes for %s\n", |
brian m. carlson | e910bb1 | 2016-09-05 20:08:01 +0000 | [diff] [blame] | 451 | oid_to_hex(&p->obj)); |
brian m. carlson | 5ee8a95 | 2017-05-30 10:30:43 -0700 | [diff] [blame] | 452 | if (add_note(t, &p->obj, &p->remote, combine_notes_overwrite)) |
Johannes Schindelin | 033abf9 | 2018-05-02 11:38:39 +0200 | [diff] [blame] | 453 | BUG("combine_notes_overwrite failed"); |
Johan Herland | 3228e67 | 2010-11-15 00:55:12 +0100 | [diff] [blame] | 454 | return 0; |
| 455 | case NOTES_MERGE_RESOLVE_UNION: |
Jonathan Nieder | 5f9f8d1 | 2011-11-17 19:27:46 -0600 | [diff] [blame] | 456 | if (o->verbosity >= 2) |
| 457 | printf("Concatenating local and remote notes for %s\n", |
brian m. carlson | e910bb1 | 2016-09-05 20:08:01 +0000 | [diff] [blame] | 458 | oid_to_hex(&p->obj)); |
brian m. carlson | 5ee8a95 | 2017-05-30 10:30:43 -0700 | [diff] [blame] | 459 | if (add_note(t, &p->obj, &p->remote, combine_notes_concatenate)) |
Johan Herland | 3228e67 | 2010-11-15 00:55:12 +0100 | [diff] [blame] | 460 | die("failed to concatenate notes " |
| 461 | "(combine_notes_concatenate)"); |
| 462 | return 0; |
Johan Herland | a6a0909 | 2010-11-15 00:57:17 +0100 | [diff] [blame] | 463 | case NOTES_MERGE_RESOLVE_CAT_SORT_UNIQ: |
Jonathan Nieder | 5f9f8d1 | 2011-11-17 19:27:46 -0600 | [diff] [blame] | 464 | if (o->verbosity >= 2) |
| 465 | printf("Concatenating unique lines in local and remote " |
brian m. carlson | e910bb1 | 2016-09-05 20:08:01 +0000 | [diff] [blame] | 466 | "notes for %s\n", oid_to_hex(&p->obj)); |
brian m. carlson | 5ee8a95 | 2017-05-30 10:30:43 -0700 | [diff] [blame] | 467 | if (add_note(t, &p->obj, &p->remote, combine_notes_cat_sort_uniq)) |
Johan Herland | a6a0909 | 2010-11-15 00:57:17 +0100 | [diff] [blame] | 468 | die("failed to concatenate notes " |
| 469 | "(combine_notes_cat_sort_uniq)"); |
| 470 | return 0; |
Johan Herland | 3228e67 | 2010-11-15 00:55:12 +0100 | [diff] [blame] | 471 | } |
| 472 | die("Unknown strategy (%i).", o->strategy); |
| 473 | } |
| 474 | |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 475 | static int merge_changes(struct notes_merge_options *o, |
| 476 | struct notes_merge_pair *changes, int *num_changes, |
| 477 | struct notes_tree *t) |
| 478 | { |
| 479 | int i, conflicts = 0; |
| 480 | |
| 481 | trace_printf("\tmerge_changes(num_changes = %i)\n", *num_changes); |
| 482 | for (i = 0; i < *num_changes; i++) { |
| 483 | struct notes_merge_pair *p = changes + i; |
| 484 | trace_printf("\t\t%.7s: %.7s -> %.7s/%.7s\n", |
brian m. carlson | e910bb1 | 2016-09-05 20:08:01 +0000 | [diff] [blame] | 485 | oid_to_hex(&p->obj), oid_to_hex(&p->base), |
| 486 | oid_to_hex(&p->local), |
| 487 | oid_to_hex(&p->remote)); |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 488 | |
Jeff King | 4a7e27e | 2018-08-28 17:22:40 -0400 | [diff] [blame] | 489 | if (oideq(&p->base, &p->remote)) { |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 490 | /* no remote change; nothing to do */ |
| 491 | trace_printf("\t\t\tskipping (no remote change)\n"); |
Jeff King | 4a7e27e | 2018-08-28 17:22:40 -0400 | [diff] [blame] | 492 | } else if (oideq(&p->local, &p->remote)) { |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 493 | /* same change in local and remote; nothing to do */ |
| 494 | trace_printf("\t\t\tskipping (local == remote)\n"); |
Jeff King | 4a7e27e | 2018-08-28 17:22:40 -0400 | [diff] [blame] | 495 | } else if (oideq(&p->local, &uninitialized) || |
| 496 | oideq(&p->local, &p->base)) { |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 497 | /* no local change; adopt remote change */ |
| 498 | trace_printf("\t\t\tno local change, adopted remote\n"); |
brian m. carlson | 5ee8a95 | 2017-05-30 10:30:43 -0700 | [diff] [blame] | 499 | if (add_note(t, &p->obj, &p->remote, |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 500 | combine_notes_overwrite)) |
Johannes Schindelin | 033abf9 | 2018-05-02 11:38:39 +0200 | [diff] [blame] | 501 | BUG("combine_notes_overwrite failed"); |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 502 | } else { |
| 503 | /* need file-level merge between local and remote */ |
| 504 | trace_printf("\t\t\tneed content-level merge\n"); |
Johan Herland | 3228e67 | 2010-11-15 00:55:12 +0100 | [diff] [blame] | 505 | conflicts += merge_one_change(o, p, t); |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 506 | } |
| 507 | } |
| 508 | |
| 509 | return conflicts; |
| 510 | } |
| 511 | |
| 512 | static int merge_from_diffs(struct notes_merge_options *o, |
Brandon Williams | 9d6babb | 2017-05-30 10:30:59 -0700 | [diff] [blame] | 513 | const struct object_id *base, |
| 514 | const struct object_id *local, |
| 515 | const struct object_id *remote, |
| 516 | struct notes_tree *t) |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 517 | { |
| 518 | struct notes_merge_pair *changes; |
| 519 | int num_changes, conflicts; |
| 520 | |
| 521 | trace_printf("\tmerge_from_diffs(base = %.7s, local = %.7s, " |
Brandon Williams | 9d6babb | 2017-05-30 10:30:59 -0700 | [diff] [blame] | 522 | "remote = %.7s)\n", oid_to_hex(base), oid_to_hex(local), |
| 523 | oid_to_hex(remote)); |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 524 | |
| 525 | changes = diff_tree_remote(o, base, remote, &num_changes); |
| 526 | diff_tree_local(o, changes, num_changes, base, local); |
| 527 | |
| 528 | conflicts = merge_changes(o, changes, &num_changes, t); |
| 529 | free(changes); |
| 530 | |
Jonathan Nieder | 5f9f8d1 | 2011-11-17 19:27:46 -0600 | [diff] [blame] | 531 | if (o->verbosity >= 4) |
Nguyễn Thái Ngọc Duy | 2ca0c53 | 2012-06-07 19:05:13 +0700 | [diff] [blame] | 532 | printf(t->dirty ? |
| 533 | "Merge result: %i unmerged notes and a dirty notes tree\n" : |
| 534 | "Merge result: %i unmerged notes and a clean notes tree\n", |
| 535 | conflicts); |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 536 | |
| 537 | return conflicts ? -1 : 1; |
| 538 | } |
| 539 | |
Johan Herland | 75ef3f4 | 2010-11-09 22:49:46 +0100 | [diff] [blame] | 540 | int notes_merge(struct notes_merge_options *o, |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 541 | struct notes_tree *local_tree, |
Brandon Williams | 5237e0e | 2017-05-30 10:30:58 -0700 | [diff] [blame] | 542 | struct object_id *result_oid) |
Johan Herland | 75ef3f4 | 2010-11-09 22:49:46 +0100 | [diff] [blame] | 543 | { |
brian m. carlson | 1e43ed9 | 2017-05-06 22:10:09 +0000 | [diff] [blame] | 544 | struct object_id local_oid, remote_oid; |
Johan Herland | 75ef3f4 | 2010-11-09 22:49:46 +0100 | [diff] [blame] | 545 | struct commit *local, *remote; |
| 546 | struct commit_list *bases = NULL; |
Brandon Williams | 5237e0e | 2017-05-30 10:30:58 -0700 | [diff] [blame] | 547 | const struct object_id *base_oid, *base_tree_oid; |
Johan Herland | 75ef3f4 | 2010-11-09 22:49:46 +0100 | [diff] [blame] | 548 | int result = 0; |
| 549 | |
| 550 | assert(o->local_ref && o->remote_ref); |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 551 | assert(!strcmp(o->local_ref, local_tree->ref)); |
Brandon Williams | 5237e0e | 2017-05-30 10:30:58 -0700 | [diff] [blame] | 552 | oidclr(result_oid); |
Johan Herland | 75ef3f4 | 2010-11-09 22:49:46 +0100 | [diff] [blame] | 553 | |
| 554 | trace_printf("notes_merge(o->local_ref = %s, o->remote_ref = %s)\n", |
| 555 | o->local_ref, o->remote_ref); |
| 556 | |
| 557 | /* Dereference o->local_ref into local_sha1 */ |
brian m. carlson | 34c290a | 2017-10-15 22:06:56 +0000 | [diff] [blame] | 558 | if (read_ref_full(o->local_ref, 0, &local_oid, NULL)) |
Johan Herland | 75ef3f4 | 2010-11-09 22:49:46 +0100 | [diff] [blame] | 559 | die("Failed to resolve local notes ref '%s'", o->local_ref); |
Michael Haggerty | 8d9c501 | 2011-09-15 23:10:25 +0200 | [diff] [blame] | 560 | else if (!check_refname_format(o->local_ref, 0) && |
brian m. carlson | 1e43ed9 | 2017-05-06 22:10:09 +0000 | [diff] [blame] | 561 | is_null_oid(&local_oid)) |
Brandon Williams | 5237e0e | 2017-05-30 10:30:58 -0700 | [diff] [blame] | 562 | local = NULL; /* local_oid == null_oid indicates unborn ref */ |
Nguyễn Thái Ngọc Duy | 878d832 | 2018-11-10 06:48:54 +0100 | [diff] [blame] | 563 | else if (!(local = lookup_commit_reference(o->repo, &local_oid))) |
Johan Herland | 75ef3f4 | 2010-11-09 22:49:46 +0100 | [diff] [blame] | 564 | die("Could not parse local commit %s (%s)", |
brian m. carlson | 1e43ed9 | 2017-05-06 22:10:09 +0000 | [diff] [blame] | 565 | oid_to_hex(&local_oid), o->local_ref); |
| 566 | trace_printf("\tlocal commit: %.7s\n", oid_to_hex(&local_oid)); |
Johan Herland | 75ef3f4 | 2010-11-09 22:49:46 +0100 | [diff] [blame] | 567 | |
Brandon Williams | 5237e0e | 2017-05-30 10:30:58 -0700 | [diff] [blame] | 568 | /* Dereference o->remote_ref into remote_oid */ |
brian m. carlson | 1e43ed9 | 2017-05-06 22:10:09 +0000 | [diff] [blame] | 569 | if (get_oid(o->remote_ref, &remote_oid)) { |
Johan Herland | 75ef3f4 | 2010-11-09 22:49:46 +0100 | [diff] [blame] | 570 | /* |
Brandon Williams | 5237e0e | 2017-05-30 10:30:58 -0700 | [diff] [blame] | 571 | * Failed to get remote_oid. If o->remote_ref looks like an |
Johan Herland | 75ef3f4 | 2010-11-09 22:49:46 +0100 | [diff] [blame] | 572 | * unborn ref, perform the merge using an empty notes tree. |
| 573 | */ |
Michael Haggerty | 8d9c501 | 2011-09-15 23:10:25 +0200 | [diff] [blame] | 574 | if (!check_refname_format(o->remote_ref, 0)) { |
brian m. carlson | 1e43ed9 | 2017-05-06 22:10:09 +0000 | [diff] [blame] | 575 | oidclr(&remote_oid); |
Johan Herland | 75ef3f4 | 2010-11-09 22:49:46 +0100 | [diff] [blame] | 576 | remote = NULL; |
| 577 | } else { |
| 578 | die("Failed to resolve remote notes ref '%s'", |
| 579 | o->remote_ref); |
| 580 | } |
Nguyễn Thái Ngọc Duy | 878d832 | 2018-11-10 06:48:54 +0100 | [diff] [blame] | 581 | } else if (!(remote = lookup_commit_reference(o->repo, &remote_oid))) { |
Johan Herland | 75ef3f4 | 2010-11-09 22:49:46 +0100 | [diff] [blame] | 582 | die("Could not parse remote commit %s (%s)", |
brian m. carlson | 1e43ed9 | 2017-05-06 22:10:09 +0000 | [diff] [blame] | 583 | oid_to_hex(&remote_oid), o->remote_ref); |
Johan Herland | 75ef3f4 | 2010-11-09 22:49:46 +0100 | [diff] [blame] | 584 | } |
brian m. carlson | 1e43ed9 | 2017-05-06 22:10:09 +0000 | [diff] [blame] | 585 | trace_printf("\tremote commit: %.7s\n", oid_to_hex(&remote_oid)); |
Johan Herland | 75ef3f4 | 2010-11-09 22:49:46 +0100 | [diff] [blame] | 586 | |
| 587 | if (!local && !remote) |
| 588 | die("Cannot merge empty notes ref (%s) into empty notes ref " |
| 589 | "(%s)", o->remote_ref, o->local_ref); |
| 590 | if (!local) { |
| 591 | /* result == remote commit */ |
Brandon Williams | 5237e0e | 2017-05-30 10:30:58 -0700 | [diff] [blame] | 592 | oidcpy(result_oid, &remote_oid); |
Johan Herland | 75ef3f4 | 2010-11-09 22:49:46 +0100 | [diff] [blame] | 593 | goto found_result; |
| 594 | } |
| 595 | if (!remote) { |
| 596 | /* result == local commit */ |
Brandon Williams | 5237e0e | 2017-05-30 10:30:58 -0700 | [diff] [blame] | 597 | oidcpy(result_oid, &local_oid); |
Johan Herland | 75ef3f4 | 2010-11-09 22:49:46 +0100 | [diff] [blame] | 598 | goto found_result; |
| 599 | } |
| 600 | assert(local && remote); |
| 601 | |
| 602 | /* Find merge bases */ |
Junio C Hamano | 2ce406c | 2014-10-30 12:20:44 -0700 | [diff] [blame] | 603 | bases = get_merge_bases(local, remote); |
Johan Herland | 75ef3f4 | 2010-11-09 22:49:46 +0100 | [diff] [blame] | 604 | if (!bases) { |
brian m. carlson | 1422844 | 2021-04-26 01:02:56 +0000 | [diff] [blame] | 605 | base_oid = null_oid(); |
brian m. carlson | eb0ccfd | 2017-11-12 21:28:54 +0000 | [diff] [blame] | 606 | base_tree_oid = the_hash_algo->empty_tree; |
Jonathan Nieder | 5f9f8d1 | 2011-11-17 19:27:46 -0600 | [diff] [blame] | 607 | if (o->verbosity >= 4) |
| 608 | printf("No merge base found; doing history-less merge\n"); |
Johan Herland | 75ef3f4 | 2010-11-09 22:49:46 +0100 | [diff] [blame] | 609 | } else if (!bases->next) { |
Brandon Williams | 5237e0e | 2017-05-30 10:30:58 -0700 | [diff] [blame] | 610 | base_oid = &bases->item->object.oid; |
Derrick Stolee | 2e27bd7 | 2018-04-06 19:09:38 +0000 | [diff] [blame] | 611 | base_tree_oid = get_commit_tree_oid(bases->item); |
Jonathan Nieder | 5f9f8d1 | 2011-11-17 19:27:46 -0600 | [diff] [blame] | 612 | if (o->verbosity >= 4) |
| 613 | printf("One merge base found (%.7s)\n", |
Brandon Williams | 5237e0e | 2017-05-30 10:30:58 -0700 | [diff] [blame] | 614 | oid_to_hex(base_oid)); |
Johan Herland | 75ef3f4 | 2010-11-09 22:49:46 +0100 | [diff] [blame] | 615 | } else { |
| 616 | /* TODO: How to handle multiple merge-bases? */ |
Brandon Williams | 5237e0e | 2017-05-30 10:30:58 -0700 | [diff] [blame] | 617 | base_oid = &bases->item->object.oid; |
Derrick Stolee | 2e27bd7 | 2018-04-06 19:09:38 +0000 | [diff] [blame] | 618 | base_tree_oid = get_commit_tree_oid(bases->item); |
Jonathan Nieder | 5f9f8d1 | 2011-11-17 19:27:46 -0600 | [diff] [blame] | 619 | if (o->verbosity >= 3) |
| 620 | printf("Multiple merge bases found. Using the first " |
Brandon Williams | 5237e0e | 2017-05-30 10:30:58 -0700 | [diff] [blame] | 621 | "(%.7s)\n", oid_to_hex(base_oid)); |
Johan Herland | 75ef3f4 | 2010-11-09 22:49:46 +0100 | [diff] [blame] | 622 | } |
| 623 | |
Jonathan Nieder | 5f9f8d1 | 2011-11-17 19:27:46 -0600 | [diff] [blame] | 624 | if (o->verbosity >= 4) |
| 625 | printf("Merging remote commit %.7s into local commit %.7s with " |
brian m. carlson | f2fd076 | 2015-11-10 02:22:28 +0000 | [diff] [blame] | 626 | "merge-base %.7s\n", oid_to_hex(&remote->object.oid), |
| 627 | oid_to_hex(&local->object.oid), |
Brandon Williams | 5237e0e | 2017-05-30 10:30:58 -0700 | [diff] [blame] | 628 | oid_to_hex(base_oid)); |
Johan Herland | 75ef3f4 | 2010-11-09 22:49:46 +0100 | [diff] [blame] | 629 | |
Jeff King | 4a7e27e | 2018-08-28 17:22:40 -0400 | [diff] [blame] | 630 | if (oideq(&remote->object.oid, base_oid)) { |
Johan Herland | 75ef3f4 | 2010-11-09 22:49:46 +0100 | [diff] [blame] | 631 | /* Already merged; result == local commit */ |
Jonathan Nieder | 5f9f8d1 | 2011-11-17 19:27:46 -0600 | [diff] [blame] | 632 | if (o->verbosity >= 2) |
Eric Sunshine | 80cde95 | 2021-05-02 01:14:22 -0400 | [diff] [blame] | 633 | printf_ln("Already up to date."); |
Brandon Williams | 5237e0e | 2017-05-30 10:30:58 -0700 | [diff] [blame] | 634 | oidcpy(result_oid, &local->object.oid); |
Johan Herland | 75ef3f4 | 2010-11-09 22:49:46 +0100 | [diff] [blame] | 635 | goto found_result; |
| 636 | } |
Jeff King | 4a7e27e | 2018-08-28 17:22:40 -0400 | [diff] [blame] | 637 | if (oideq(&local->object.oid, base_oid)) { |
Johan Herland | 75ef3f4 | 2010-11-09 22:49:46 +0100 | [diff] [blame] | 638 | /* Fast-forward; result == remote commit */ |
Jonathan Nieder | 5f9f8d1 | 2011-11-17 19:27:46 -0600 | [diff] [blame] | 639 | if (o->verbosity >= 2) |
| 640 | printf("Fast-forward\n"); |
Brandon Williams | 5237e0e | 2017-05-30 10:30:58 -0700 | [diff] [blame] | 641 | oidcpy(result_oid, &remote->object.oid); |
Johan Herland | 75ef3f4 | 2010-11-09 22:49:46 +0100 | [diff] [blame] | 642 | goto found_result; |
| 643 | } |
| 644 | |
Derrick Stolee | 2e27bd7 | 2018-04-06 19:09:38 +0000 | [diff] [blame] | 645 | result = merge_from_diffs(o, base_tree_oid, |
| 646 | get_commit_tree_oid(local), |
| 647 | get_commit_tree_oid(remote), local_tree); |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 648 | |
Johan Herland | 809f38c | 2010-11-09 22:49:51 +0100 | [diff] [blame] | 649 | if (result != 0) { /* non-trivial merge (with or without conflicts) */ |
| 650 | /* Commit (partial) result */ |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 651 | struct commit_list *parents = NULL; |
| 652 | commit_list_insert(remote, &parents); /* LIFO order */ |
| 653 | commit_list_insert(local, &parents); |
Nguyễn Thái Ngọc Duy | 1d18d75 | 2019-01-12 09:13:23 +0700 | [diff] [blame] | 654 | create_notes_commit(o->repo, local_tree, parents, o->commit_msg.buf, |
Patryk Obara | 5078f34 | 2018-01-28 01:13:16 +0100 | [diff] [blame] | 655 | o->commit_msg.len, result_oid); |
Johan Herland | 2085b16 | 2010-11-15 00:54:11 +0100 | [diff] [blame] | 656 | } |
Johan Herland | 75ef3f4 | 2010-11-09 22:49:46 +0100 | [diff] [blame] | 657 | |
| 658 | found_result: |
| 659 | free_commit_list(bases); |
Johan Herland | 443259c | 2010-11-09 22:49:53 +0100 | [diff] [blame] | 660 | strbuf_release(&(o->commit_msg)); |
Brandon Williams | 5237e0e | 2017-05-30 10:30:58 -0700 | [diff] [blame] | 661 | trace_printf("notes_merge(): result = %i, result_oid = %.7s\n", |
| 662 | result, oid_to_hex(result_oid)); |
Johan Herland | 75ef3f4 | 2010-11-09 22:49:46 +0100 | [diff] [blame] | 663 | return result; |
| 664 | } |
Johan Herland | 6abb365 | 2010-11-09 22:49:52 +0100 | [diff] [blame] | 665 | |
| 666 | int notes_merge_commit(struct notes_merge_options *o, |
| 667 | struct notes_tree *partial_tree, |
| 668 | struct commit *partial_commit, |
Brandon Williams | 5237e0e | 2017-05-30 10:30:58 -0700 | [diff] [blame] | 669 | struct object_id *result_oid) |
Johan Herland | 6abb365 | 2010-11-09 22:49:52 +0100 | [diff] [blame] | 670 | { |
| 671 | /* |
| 672 | * Iterate through files in .git/NOTES_MERGE_WORKTREE and add all |
Johan Herland | a0be62c | 2012-03-12 15:57:13 +0100 | [diff] [blame] | 673 | * found notes to 'partial_tree'. Write the updated notes tree to |
Johan Herland | 6abb365 | 2010-11-09 22:49:52 +0100 | [diff] [blame] | 674 | * the DB, and commit the resulting tree object while reusing the |
| 675 | * commit message and parents from 'partial_commit'. |
Brandon Williams | 5237e0e | 2017-05-30 10:30:58 -0700 | [diff] [blame] | 676 | * Finally store the new commit object OID into 'result_oid'. |
Johan Herland | 6abb365 | 2010-11-09 22:49:52 +0100 | [diff] [blame] | 677 | */ |
Johan Herland | a0be62c | 2012-03-12 15:57:13 +0100 | [diff] [blame] | 678 | DIR *dir; |
| 679 | struct dirent *e; |
| 680 | struct strbuf path = STRBUF_INIT; |
Jeff King | 8597ea3 | 2014-06-10 17:44:13 -0400 | [diff] [blame] | 681 | const char *buffer = get_commit_buffer(partial_commit, NULL); |
Jeff King | bc6b8fc | 2014-06-10 17:41:51 -0400 | [diff] [blame] | 682 | const char *msg = strstr(buffer, "\n\n"); |
Johan Herland | a0be62c | 2012-03-12 15:57:13 +0100 | [diff] [blame] | 683 | int baselen; |
Johan Herland | 6abb365 | 2010-11-09 22:49:52 +0100 | [diff] [blame] | 684 | |
Jeff King | 8c2ca3a | 2017-04-20 17:09:30 -0400 | [diff] [blame] | 685 | git_path_buf(&path, NOTES_MERGE_WORKTREE); |
Jonathan Nieder | 5f9f8d1 | 2011-11-17 19:27:46 -0600 | [diff] [blame] | 686 | if (o->verbosity >= 3) |
Johan Herland | a0be62c | 2012-03-12 15:57:13 +0100 | [diff] [blame] | 687 | printf("Committing notes in notes merge worktree at %s\n", |
| 688 | path.buf); |
Johan Herland | 6abb365 | 2010-11-09 22:49:52 +0100 | [diff] [blame] | 689 | |
| 690 | if (!msg || msg[2] == '\0') |
| 691 | die("partial notes commit has empty message"); |
| 692 | msg += 2; |
| 693 | |
Johan Herland | a0be62c | 2012-03-12 15:57:13 +0100 | [diff] [blame] | 694 | dir = opendir(path.buf); |
| 695 | if (!dir) |
| 696 | die_errno("could not open %s", path.buf); |
| 697 | |
| 698 | strbuf_addch(&path, '/'); |
| 699 | baselen = path.len; |
Elijah Newren | b548f0f | 2021-05-12 17:28:22 +0000 | [diff] [blame] | 700 | while ((e = readdir_skip_dot_and_dotdot(dir)) != NULL) { |
Johan Herland | 6abb365 | 2010-11-09 22:49:52 +0100 | [diff] [blame] | 701 | struct stat st; |
brian m. carlson | 5ee8a95 | 2017-05-30 10:30:43 -0700 | [diff] [blame] | 702 | struct object_id obj_oid, blob_oid; |
Johan Herland | 6abb365 | 2010-11-09 22:49:52 +0100 | [diff] [blame] | 703 | |
brian m. carlson | 5ee8a95 | 2017-05-30 10:30:43 -0700 | [diff] [blame] | 704 | if (get_oid_hex(e->d_name, &obj_oid)) { |
Jonathan Nieder | 5f9f8d1 | 2011-11-17 19:27:46 -0600 | [diff] [blame] | 705 | if (o->verbosity >= 3) |
Johan Herland | a0be62c | 2012-03-12 15:57:13 +0100 | [diff] [blame] | 706 | printf("Skipping non-SHA1 entry '%s%s'\n", |
| 707 | path.buf, e->d_name); |
Johan Herland | 6abb365 | 2010-11-09 22:49:52 +0100 | [diff] [blame] | 708 | continue; |
| 709 | } |
| 710 | |
Johan Herland | a0be62c | 2012-03-12 15:57:13 +0100 | [diff] [blame] | 711 | strbuf_addstr(&path, e->d_name); |
Johan Herland | 6abb365 | 2010-11-09 22:49:52 +0100 | [diff] [blame] | 712 | /* write file as blob, and add to partial_tree */ |
Johan Herland | a0be62c | 2012-03-12 15:57:13 +0100 | [diff] [blame] | 713 | if (stat(path.buf, &st)) |
| 714 | die_errno("Failed to stat '%s'", path.buf); |
Nguyễn Thái Ngọc Duy | 5684200 | 2018-11-10 06:48:53 +0100 | [diff] [blame] | 715 | if (index_path(o->repo->index, &blob_oid, path.buf, &st, HASH_WRITE_OBJECT)) |
Johan Herland | a0be62c | 2012-03-12 15:57:13 +0100 | [diff] [blame] | 716 | die("Failed to write blob object from '%s'", path.buf); |
brian m. carlson | 5ee8a95 | 2017-05-30 10:30:43 -0700 | [diff] [blame] | 717 | if (add_note(partial_tree, &obj_oid, &blob_oid, NULL)) |
Johan Herland | 6abb365 | 2010-11-09 22:49:52 +0100 | [diff] [blame] | 718 | die("Failed to add resolved note '%s' to notes tree", |
Johan Herland | a0be62c | 2012-03-12 15:57:13 +0100 | [diff] [blame] | 719 | path.buf); |
Jonathan Nieder | 5f9f8d1 | 2011-11-17 19:27:46 -0600 | [diff] [blame] | 720 | if (o->verbosity >= 4) |
| 721 | printf("Added resolved note for object %s: %s\n", |
brian m. carlson | 5ee8a95 | 2017-05-30 10:30:43 -0700 | [diff] [blame] | 722 | oid_to_hex(&obj_oid), oid_to_hex(&blob_oid)); |
Johan Herland | a0be62c | 2012-03-12 15:57:13 +0100 | [diff] [blame] | 723 | strbuf_setlen(&path, baselen); |
Johan Herland | 6abb365 | 2010-11-09 22:49:52 +0100 | [diff] [blame] | 724 | } |
| 725 | |
Nguyễn Thái Ngọc Duy | 1d18d75 | 2019-01-12 09:13:23 +0700 | [diff] [blame] | 726 | create_notes_commit(o->repo, partial_tree, partial_commit->parents, msg, |
Patryk Obara | 5078f34 | 2018-01-28 01:13:16 +0100 | [diff] [blame] | 727 | strlen(msg), result_oid); |
Jeff King | bc6b8fc | 2014-06-10 17:41:51 -0400 | [diff] [blame] | 728 | unuse_commit_buffer(partial_commit, buffer); |
Jonathan Nieder | 5f9f8d1 | 2011-11-17 19:27:46 -0600 | [diff] [blame] | 729 | if (o->verbosity >= 4) |
| 730 | printf("Finalized notes merge commit: %s\n", |
Brandon Williams | 5237e0e | 2017-05-30 10:30:58 -0700 | [diff] [blame] | 731 | oid_to_hex(result_oid)); |
Johan Herland | a0be62c | 2012-03-12 15:57:13 +0100 | [diff] [blame] | 732 | strbuf_release(&path); |
| 733 | closedir(dir); |
Johan Herland | 6abb365 | 2010-11-09 22:49:52 +0100 | [diff] [blame] | 734 | return 0; |
| 735 | } |
| 736 | |
| 737 | int notes_merge_abort(struct notes_merge_options *o) |
| 738 | { |
Johan Herland | dabba59 | 2012-03-15 15:58:56 +0100 | [diff] [blame] | 739 | /* |
| 740 | * Remove all files within .git/NOTES_MERGE_WORKTREE. We do not remove |
| 741 | * the .git/NOTES_MERGE_WORKTREE directory itself, since it might be |
| 742 | * the current working directory of the user. |
| 743 | */ |
Johan Herland | 6abb365 | 2010-11-09 22:49:52 +0100 | [diff] [blame] | 744 | struct strbuf buf = STRBUF_INIT; |
| 745 | int ret; |
| 746 | |
Jeff King | 8c2ca3a | 2017-04-20 17:09:30 -0400 | [diff] [blame] | 747 | git_path_buf(&buf, NOTES_MERGE_WORKTREE); |
Jonathan Nieder | 5f9f8d1 | 2011-11-17 19:27:46 -0600 | [diff] [blame] | 748 | if (o->verbosity >= 3) |
Johan Herland | dabba59 | 2012-03-15 15:58:56 +0100 | [diff] [blame] | 749 | printf("Removing notes merge worktree at %s/*\n", buf.buf); |
| 750 | ret = remove_dir_recursively(&buf, REMOVE_DIR_KEEP_TOPLEVEL); |
Johan Herland | 6abb365 | 2010-11-09 22:49:52 +0100 | [diff] [blame] | 751 | strbuf_release(&buf); |
| 752 | return ret; |
| 753 | } |