Thread (5 messages) flat view 5 messages, 2 authors, 2016-06-15

Re: [PATCH 1/4] Teach git mergetool to use custom commands defined at config time

From: Junio C Hamano <hidden>
Date: 2016-06-15 22:44:14

Charles Bailey [off-list ref] writes:
This series of patches are a cleaned and improved version of the
earlier RFC patch that I sent.
About the organization of the series, I think (note that what I
think does not count as much as what Ted thinks around this
area) it would make much more sense to do 4 first (unless there
is a reason why it behaves differently for existing backends),
then 2 (remove $path everywhere and use $MERGED consistently,
not just where you call out to the custom tool), and then
finally 1.  The general idea is to clean-up first and then add
features on solidified base.

I do not personally see much need for 3, as the custom script
should be able to check the situation and adjust its behaviour
accordingly, and that way you do not have to maintain two
scripts.  I.e. if $BASE does not exist, there is no base, no?
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help