Re: Cogito: cg-clone doesn't like packed tag objects

From: Junio C Hamano <junkio@cox.net>
Date: Sep 28, 2005, 17:22 UTC
Message-ID: <7virwlumyo.fsf@assigned-by-dhcp.cox.net>
In-Reply-To: <Pine.LNX.4.58.0509271414000.3308@g5.osdl.org>

Linus Torvalds <torvalds@osdl.org> writes:

Show 8 quoted lines

>> we could send phony entries like this:
>> 
>> b92c9c07fe2d0d89c4f692573583c4753b5355d2	deref/tags/junio-gpg-pub
>> a3eb250f996bf5e12376ec88622c4ccaabf20ea8	deref/tags/v0.99
>> 78d9d414123ad6f4f522ffecbcd9e4a7562948fd	deref/tags/v0.99.1
>
> Yes, it would work,..
> in general, it's just a really ugly special case, I think.

I think we could do this instead, to make it less ugly.

b92c9c07fe2d0d89c4f692573583c4753b5355d2 refs/tags/junio-gpg-pub!* a3eb250f996bf5e12376ec88622c4ccaabf20ea8 refs/tags/v0.99^0

I am not sure what the syntax should be, but the idea is to express the "refname" side using "extended SHA1" syntax. In the above example, I added a postfix '!*' to mean "deref tag zero-or-more times until you get a non-tag" ('!' to mean deref tag once and complain if the object is not tag, '!!' is deref twice, '!!!' is to deref three times and so on). It might be better to spell "v0.99^0" as "v0.99!*" in this context. [*1*]

Both git-clone-pack and git-fetch-pack need to be told to ignore funny tagnames with trailing '!*', otherwise they would ask for the pointed-at object (which is not harmful but redundant) and the clone would create "refs/tags/v0.99!*", a file with a funny name. Git-peek-remote should report that, and server-info.c should be told to prepare these extra entries for ls-remote over other protocols.

But I tend to agree that this is really a special case needed to support the "tagged objects are automatically followed by tags that tag them" model, and not needed if we stay in "tag is just a ref, and a ref is just an object name, and asking for an object pulls in other objects that are reachable from them" model. So it is not a very high priority for me, but I think this is one way to help Cogito cleanly, and I am willing to see how much damage this would cause to other parts of the core, *if* Cogito wants to use this mechanism.

The alternative would be what Pasky outlined in his message -- bypassing git transport layer to fetch single object by hand, repeatedly dereferencing it until he gets a non-tag. I think that is unnecessary misery for him.

[Footnote]

*1* The difference from '^0' is that '!' does not complain on non-commit, and can be used to peel the onion one layer at a time. I do not know how useful the latter is in practice but somebody may want to express chains of trust by signing tags.