On Sat, Jan 08, 2011 at 12:21:33AM +0200, Ilari Liusvaara wrote: > On Fri, Jan 07, 2011 at 04:56:31PM -0500, Jeff King wrote: > > On Fri, Jan 07, 2011 at 11:45:01PM +0200, Ilari Liusvaara wrote: > > > > > > I think there should always be an assumption that mirrors are not > > necessarily complete. That is necessary for bundle-like mirrors to be > > feasible, since updating the bundle for every commit defeats the > > purpose. > > Also add protocol that grabs a bundle from HTTP and then opens that > up? :-) Well, yes, that still needs to be implemented. But it's all client-side, so the server just has to provide the bundle somewhere. > > It would be nice for there to be a way for some mirrors to be marked as > > "should be considered complete and authoritative", since we can optimize > > out the final check of the master in that case (as well as for future > > fetches). But that's a future feature. My plan was to leave space in the > > mirror list for arbitrary metadata of that sort. > > The first thing one should get/do when connecting to another repository > is its list of references. One can see from there if what one has got > is complete or not (with --use-mirror that only allows skipping commit > negotiation and fetch, not the whole connection due to the fact that the > repositories are contacted in order)... Yes, but it would be cool to be able to skip even that connect in some cases (e.g., mirrors can be useful not just to take load off the master, but also when the master isn't available, either for downtime or because the client is behind a firewall). But the default should definitely be to double-check that the master is right, and we can leave more advanced cases for later (we just need to be aware of leaving room for them now). I'm going to start working on a patch series for this, so hopefully we'll see how it's shaping up in a day or two. -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