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 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, 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  http://article.gmane.org/gmane.comp.version-control.git/188387  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 majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html