From: Sverre Rabbelier <srabbelier@gmail.com>
Date: Wed, 08 Jun 2011 20:52:22 GMT
Subject: Re: [PATCH 16/19] transport-helper: change import semantics
Message-ID: <BANLkTi=tfE2GtpSZxY7ir98MDREefSFT1g@mail.gmail.com>
In-Reply-To: <20110608204754.GK27715@elie>

Heya,

On Wed, Jun 8, 2011 at 22:47, Jonathan Nieder <jrnieder@gmail.com> wrote:
> Wait, now that I look there, wouldn't it be more consistent with
> what "push" does to make the semantics
>
>        ... A batch sequence of
>        one or more import commands is terminated with a blank line.
>
> ?  In other words, what was the rationale for making push work like
>
>        push +src1:dst1
>        push +src2:dst2
>        push +src3:dst3
>
>        [... command stream continues here ...]
>
> instead of
>
>        push
>        +src1:dst1
>        +src2:dst2
>        +src3:dst3
>
>        [... command stream continues ...]
>
> and does it apply for import, too?

I don't know. Shawn added the push capability in ae4efe19 (Move WebDAV
HTTP push under remote-curl, Fri Oct 30 2009) but there is no
rationale for doing it that way there. Shawn, do you remember?

-- 
Cheers,

Sverre Rabbelier