Re: [PATCH 5/7] remote-helpers: add support for an export command

From: Daniel Barkalow <barkalow@iabervon.org>
Date: Mar 29, 2010, 17:21 UTC
Message-ID: <alpine.LNX.2.00.1003291256020.14365@iabervon.org>
In-Reply-To: <1269881309-19690-6-git-send-email-srabbelier@gmail.com>

On Mon, 29 Mar 2010, Sverre Rabbelier wrote:

Show 5 quoted lines

> ---
> 
> 	This complements the 'import' capability, making it possible
> 	to use remote-helpers with only fast-import/fast-exports
> 	streams.

It would be good to have documentation on what the helper should do doing here; it's relatively difficult to tell from this code.

I think that it's getting some options (import and export marks) back from the helper in a kind of ad-hoc and unclear fashion, and it should be doing something more appropriate. (I'm not sure what the more appropriate thing is, or really what the requirements are here as far as getting information around, but I think it needs to be discussed.)

	-Daniel
*This .sig left intentionally blank*