Re: [PATCH v4] submodule merge: update conflict error message
- From
Elijah Newren <newren@gmail.com>
- Date
- Jul 17, 2022, 02:46 UTC
- Message-ID
- <CABPp-BFHeobAdTz437CewmvbngCABTDeO4mUdHu0hGp=18UwMw@mail.gmail.com>
- In-Reply-To
- <xmqqbktsdgro.fsf@gitster.g>
On Wed, Jul 13, 2022 at 11:11 AM Junio C Hamano <gitster@pobox.com> wrote:
Show 21 quoted lines
> > Calvin Wan <calvinwan@google.com> writes: > > > Changes since v3: > > Thank you again Elijah for the helpful feedback! I have removed any code > > touching merge-recursive.c, and refactored the rest into merge-ort.c. > > The error message has been updated as well as any relevant test cases. I > > had added a jump in v3 to "ret:" in merge_submodule() to accomodate > > early returns, but this has been proven to not be necessary since an > > early return means the submodule was either renamed or deleted, and this > > case is already taken care of with the message "CONFLICT (modify/delete):" > > Unfortunately a34edae6 (merge-ort: split out a separate > display_update_messages() function, 2022-06-18) moves the code this > patch touches to a separate function. > > If this is now an ort-specific topic by dropping merge-recursive > support, perhaps we'd need to coordinate with the other branch. > > I think en/merge-tree is ready to be merged down to 'master' in a > few days, so I'll wait for [v5] of this patch after that happens.
Yeah, sorry about that Calvin; one of the risks of working in the same area. Should be pretty easy to resolve, though; let me know if you have any questions when you go to re-roll.