Re: git-rev-list: --objects list inconsistency

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

 




On Tue, 14 Nov 2006, Johannes Sixt wrote:
> 
> As you can see, without --objects, master^^..master and --max-count=2
> list the same two commits.

"--max-count=X" and "master^^..master" are two TOTALLY different things.

The fact that they happened to give the same results without "--objects" 
is a random thing, and not at all guaranteed. For example, if "master" (or 
its parent) had been a merge, it probably wouldn't have given the same 
result even _without_ "--objects".

> But with --objects I get different object lists. I don't even know which
> one is the one to expect, but I certainly would not have expected the
> lists to be different. What's wrong here?

Nothing is wrong. You're asking for two totally different things.

With "--max-count=2", you're saying "I want the first two commits", and 
then the "--objects" part extends that to all the other object types too.

So you get two commits _and_ every single object reachable from those two 
commits.

In contrast, with "master^^..master", you're saying "I want everything 
that is reachable from "master", but _not_ reachable from it's 
grand-parent. And the "--objects" flag extends that to all the other 
object types too.

So you get (in this case) two commits _and_ all the objects that are 
reachable from those two commits but that are _not_ reachable from the 
grandparent.

That's a totally different thing, obviously. There's likely to be a lot of 
objects in common between "master" and its grandparent, and you literally 
asked to not be shown those objects. In the first case, you didn't ask for 
the exclusion of the grandparent.

So you should expect a big difference in almost all cases. In one case, 
you ask for an exclusion, in the other case, you just say "just the first 
two commits, please". Not at all equivalent.

		Linus
-
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]