Re: [PATCH] Adding rebase merge strategy

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Tom Clarke <tom@...>
Cc: Carl Worth <cworth@...>, Junio C Hamano <gitster@...>, <Johannes.Schindelin@...>, <git@...>
Date: Monday, October 1, 2007 - 6:30 pm

Tom Clarke <tom@u2i.com> wrote:

`git pull -s <name>` takes any merge strategy that git-merge will
accept for its -s option.  There is also the config option of
pull.twohead that indicates what the default merge/pull strategy
should be for a two head merge.  Most people don't set this,
letting the code default to 'recursive'.

But I have to agree with (was it Junio who said this?) doing a rebase
in a merge strategy doesn't make sense when conflicts come into play.
It gets confusing fast for the end-user as the conflict resolution
process is different than for a merge.  A long time ago I wrote a
git-merge-rebase strategy and gave up on it for basically the same
reason.  I posted it to the mailing list and I think Linus said
"Why?!".  That was the end of that thread as I wound up agreeing
with him.

Multiple merge strategies can be given (and attempted).  A rebase
strategy could be attempted before recursive, and if the rebase
fails but the recursive succeeds then you get (roughly) what is
being described above by Carl.  But that still requires a rebase
merge strategy.  :-\

-- 
Shawn.
-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH] Adding rebase merge strategy, Tom Clarke, (Fri Sep 28, 12:15 pm)
Re: [PATCH] Adding rebase merge strategy, Johannes Schindelin, (Fri Sep 28, 1:03 pm)
[PATCH] Adding rebase merge strategy, Tom Clarke, (Mon Oct 1, 11:08 am)
Re: [PATCH] Adding rebase merge strategy, Junio C Hamano, (Mon Oct 1, 5:01 pm)
Re: [PATCH] Adding rebase merge strategy, Tom Clarke, (Mon Oct 1, 5:41 pm)
Re: [PATCH] Adding rebase merge strategy, Junio C Hamano, (Mon Oct 1, 6:28 pm)
Re: [PATCH] Adding rebase merge strategy, Carl Worth, (Mon Oct 1, 6:17 pm)
Re: [PATCH] Adding rebase merge strategy, J. Bruce Fields, (Mon Oct 1, 7:09 pm)
Re: [PATCH] Adding rebase merge strategy, Tom Clarke, (Mon Oct 1, 6:21 pm)
Re: [PATCH] Adding rebase merge strategy, Shawn O. Pearce, (Mon Oct 1, 6:30 pm)
Re: [PATCH] Adding rebase merge strategy, Johannes Schindelin, (Tue Oct 2, 6:00 am)
Re: [PATCH] Adding rebase merge strategy, Tom Clarke, (Tue Oct 2, 6:29 am)
Re: [PATCH] Adding rebase merge strategy, Junio C Hamano, (Tue Oct 2, 2:40 pm)
Re: [PATCH] Adding rebase merge strategy, Tom Clarke, (Wed Oct 3, 10:11 am)
Re: [PATCH] Adding rebase merge strategy, Johannes Schindelin, (Wed Oct 3, 11:54 am)
Re: [PATCH] Adding rebase merge strategy, Carl Worth, (Mon Oct 1, 6:53 pm)
Re: [PATCH] Adding rebase merge strategy, Junio C Hamano, (Mon Oct 1, 7:11 pm)
Re: [PATCH] Adding rebase merge strategy, Johannes Schindelin, (Mon Oct 1, 11:50 am)
Re: [PATCH] Adding rebase merge strategy, Tom Clarke, (Fri Sep 28, 1:18 pm)