Re: How does git now about future renames during a rebase?

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sun, May 06, 2012 at 09:41:00AM +0530, Jon Seymour wrote:

> I had a history that looked like this:
> 
> 1. some other commit
> 2. commit that moves files from one directory to a new directory
> 3. commit that edits files in the new directory.
> 
> I then did an interactive rebase to move the commit 3 before commit 2.
> 
> 1. some other commit
> 3a. commit that edits files in the new directory.
> 2a. commit that moves files from one directory to a new directory
> 
> I didn't expect this to work, but somehow git worked out that it
> needed to apply the change in 3 to the original location of the files.
> 
> How does it do this?

When you pick a commit, we actually do a merge between it and the
current HEAD, using the parent of the picked commit as a merge base. So
imagine we have this short history:

  git init repo &&
  cd repo &&
  seq 1 100 >foo && git add foo && git commit -m base &&
  git mv foo bar && git commit -m move &&
  echo 101 >>bar && git add bar && git commit -m change

I.e., a move followed by a change. And you run:

  git rebase -i HEAD~2

and swap the two commits. So afterwards we will have a change followed
by a move. What does git see?

When we pick the first commit ("change"), we end up merging with these
parameters:

  - The "ours" side is "base", with "foo" containing 1..100.
  - The "theirs" side is "change", with "foo" absent and "bar"
    containing 1..101.
  - The ancestor is "move", with "foo" absent and "bar" containing
    1..100.

So rename detection sees that our side moved bar to foo (compared to the
ancestor), and the other side added a line to bar. And it resolves by
putting the modified content into "foo". Now obviously nobody _actually_
moved bar to foo; it was quite the opposite. But that makes sense; a
cherry-pick merge like this sees the reverse of what happened between
the picked commit and the destination commit (less any changes which
have been picked already).

And then we pick the second commit ("move"), with these merge
parameters:

  - The "ours" side is now changed', "foo" containing 1..101.
  - The "theirs" side is "move", with "foo" absent and "bar" containing
    1..100.
  - The ancestor is "base", with "foo" containing 1..100.

Now it looks like their side made a move ("foo" to "bar"), and our
side modified "foo" (i.e., the opposite of the last case). So we again
resolve to put the final content in "bar".

Does that make sense?

-Peff
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]