Re: [PATCH v5 15/15] fast-export: don't handle uninteresting refs

From: Jonathan Nieder <jrnieder@gmail.com>
Date: Nov 21, 2012, 04:17 UTC
Message-ID: <20121121041735.GE4634@elie.Belkin>
In-Reply-To: <7vd2z7rj3y.fsf@alter.siamese.dyndns.org>

Junio C Hamano wrote:

> Felipe Contreras <felipe.contreras@gmail.com> writes:

Show 9 quoted lines

>> Of course, transport-helper shouldn't even be specifying the negative
>> (^) refs, but that's another story.
>
> Hrm, I am not sure I understand what you mean by this.
>
> How should it be telling the fast-export up to what commit the
> receiving end should already have the history for (hence they do not
> need to be sent)?  Or are you advocating to re-send the entire
> history down to the root commit every time?

I think Felipe has mentioned before that he considers it the remote helper's responsibility to keep track of what commits have already been imported, for example using a marks file.

Never mind that others have said that that's not the current interface (I don't yet see why it would be a good interface after a transition, but maybe it would be). Still, hopefully that clarifies the intended meaning.

Hope that helps, Jonathan