From: Jeff King Date: Tue, 07 Jun 2011 17:28:32 GMT Subject: Re: [PATCH 4/8] t5800: document some non-functional parts of remote helpers Message-ID: <20110607172832.GA23224@sigill.intra.peff.net> In-Reply-To: On Tue, Jun 07, 2011 at 07:25:49PM +0200, Sverre Rabbelier wrote: > Heya, > > On Tue, Jun 7, 2011 at 19:20, Jeff King wrote: > > There are two possible solutions: > > 3. As mentioned in a different series, I feel it would be preferable > if fast-import/fast-export did not actually modify any refs, rather > the remote-helper should specify which refs should be updated to what > value explicitly. I think the same should/could apply in this case. > Does that make sense? Isn't that the case already with import? The helper writes into a staging area (like refs/testgit), and then we let git pick the results out of there. For export, that would be the same as my (2). The problem is that we don't feed the helper _any_ information about which refs were desired. All it gets is the fast-export stream. -Peff