Re: [PATCH 07/13] transport-helper: change import semantics

From: Sverre Rabbelier <srabbelier@gmail.com>
Date: Aug 29, 2010, 20:26 UTC
Message-ID: <AANLkTi=bLe_3u+-u=+L71JiKQrQwmVz6=x=h7mws8zaC@mail.gmail.com>
In-Reply-To: <alpine.LNX.2.00.1008291521350.14365@iabervon.org>

Heya,

On Sun, Aug 29, 2010 at 14:29, Daniel Barkalow <barkalow@iabervon.org> wrote:

Show 7 quoted lines

> I think your reasons for this change could be worked around, but the
> protocol is cleaner with your change, which is justification enough, given
> that it shouldn't be too big a deal to change. This also lets the helper
> consider all of the refs it is expected to update before producing the
> stream, which may simplify the stream (particularly if the history has
> merges involving branches that may or may not be imported are aren't
> listed first).

Aye, that was also part of my motivation to do it this way (as opposed to e.g. running fast-import multiple times).

> I don't think "import" has gotten to the point where people could really
> use it in helpers not packaged with git, anyway, so I agree.

Great :)

-- 
Cheers,

Sverre Rabbelier