Re: [PATCH 2/2] remote-curl: let users turn off smart http

From: Jeff King <peff@peff.net>
Date: Sep 20, 2012, 18:12 UTC
Message-ID: <20120920181231.GA19204@sigill.intra.peff.net>
In-Reply-To: <7va9wkbmyc.fsf@alter.siamese.dyndns.org>

On Thu, Sep 20, 2012 at 10:53:15AM -0700, Junio C Hamano wrote:

Show 9 quoted lines

> Jeff King <peff@peff.net> writes:
> 
> > I added the config item as remote.foo.smarthttp. You could also allow
> > "http.$url.smart" (and just "http.smart", for that matter), which could
> > be more flexible if you have multiple remotes pointing to the same
> > broken server.
> 
> What would the user experience be when we introduce "even smarter"
> http server protocol extension?  Will we add remote.foo.starterhttp?

I would hope that it would actually be negotiated reliably at the protocol level so we do not have to deal with this mess again.

Show 6 quoted lines

> Perhaps
> 
>     remote.$name.httpvariants = [smart] [dumb]
> 
> to allow users to say "smart only", "dumb only", or "smart and/or
> dumb" might be more code but less burden on the users.

I don't mind that format if we are going that direction, but is there anybody who actually wants to say "smart only?"

-Peff