On Wed, May 29, 2013 at 12:05:41PM +0000, Ian Harvey wrote:
> So, did this patch make it anywhere? We could really use it.
>
> Here's the use case. The original ee27ca4 patch broke our build system when
> the git server was upgraded to Debian Wheezy last night. The builder fetches
> source from the repo in two pieces using git archive, and we need to make
> sure both pieces are from the same commit. So we get a sha1 hash with git
> ls-remote, and use it with git archive --remote. This, of course, breaks
> with the 'no such ref' error.
The patch you are responding to[1] would not help there, either. It does
not allow raw sha1s. The only way to do that would be:
1. Add an option to the server to allow arbitrary sha1s, even if they
are not reachable from the ref tips. This is an easy fix, but
requires server admins to cooperate (and they may or may not want
to lose the "you can only access reachable things policy".
2. Actually do a reachability check. Doing a full object check to
allow fetching an arbitrary tree by sha1 is probably prohibitively
expensive[2], but we could allow the form "<commit>[:<path>]", check
that "<commit>" is reachable, and then allow arbitrary paths within
it.
> At the very least, the documentation is wrong when it talks about passing a
> commit ID to git archive: maintainers must surely agree that the
> documentation and the actual behavior ought to match.
I am not sure which documentation you mean. The part about "commit ID"
in the current manpage is drawing the distinction between something that
resolves to a commit versus something that resolves to a tree. Either is
available both locally and remotely. I think the use of the phrase
"commit ID" is questionable there, as it really means "something that
resolves to a commit", not "a sha1 commit ID". We used to use the phrase
"commit-ish" to refer to that, but I think it has fallen out of favor as
being too jargon-y.
The documentation does not mention at all the restrictions placed on
refs using "--remote", and it probably should.
-Peff
[1] http://article.gmane.org/gmane.comp.version-control.git/188387
[2] If we had a reachability bitmap cache, calculating arbitrary object
reachability would actually be pretty cheap. But the bitmap feature
for core git is not yet ready for prime-time, so I think we should
not depend on it yet.
--
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