On Mon, Feb 26, 2018 at 4:07 PM, Johannes Schindelin <Johannes.Schindelin@xxxxxx> wrote: > Hi Buga, > > On Tue, 20 Feb 2018, Igor Djordjevic wrote: > >> I`m really interested in this topic, which seems to (try to) address the >> only "bad feeling" I had with rebasing merges - being afraid of silently >> losing amendments by actually trying to "replay" the merge (where >> additional and possibly important context is missing), instead of really >> "rebasing" it (somehow). > > If those amendments are what you are worried about, why not address them > specifically? > > In other words, rather than performing the equivalent of > > git show <merge>^! | git apply > > (which would of course completely ignore if the rewritten <merge>^2 > dropped a patch, amended a patch, or even added a new patch), what you > really want is to figure out what changes the user made when merging, and > what merge strategy was used to begin with. > > To see what I mean, look at the output of `git show 0fd90daba8`: it shows > how conflicts were resolved. By necessity, this is more complicated than a > simple diff: it is *not* as simple as taking a diff between two revisions > and applying that diff to a third revision. There were (at least) three > revisions involved in the original merge commit, and recreating that merge > commit faithfully means to represent the essence of the merge commit > faithfully enough to be able to replay it on a new set of at least three > revisions. That can be simplified to two-way diffs only in very, very > special circumstances, and in all other cases this simplification will > simply fall on its nose. > > If the proposed solution was to extend `git apply` to process combined > diffs, I would agree that we're on to something. That is not the proposed > solution, though. > > In short: while I am sympathetic to the desire to keep things simple, > the idea to somehow side-step replaying the original merge seems to be > *prone* to be flawed. Any system that cannot accommodate > dropped/changed/added commits on either side of a merge is likely to be > too limited to be useful. > The reason Sergey's solution works is because he cherry picks the merge using each parent first, and then merges the result of those. So each branch of the merge gets one, and then you merge the result of those cherry-picks. This preservers amendments and changes properly, and should result in a good solution. I agree that making "git apply" work with combined diffs could also be another solution, but it may be trickier. If this *doesn't* work, a test case showing that it doesn't work would be appreciated. I'm hoping to be able to put together something soon, but I haven't had time due to $dayjob. > Ciao, > Johannes
