Re: [PATCH] Allow cloning an empty repository

From

Sverre Rabbelier <srabbelier@gmail.com>

Date

Jan 24, 2009, 00:42 UTC

Message-ID

<bd6139dc0901231642v6bd593d3mfefaca3c419a9f0a@mail.gmail.com>

In-Reply-To

<alpine.DEB.1.00.0901240131570.3586@pacific.mpi-cbg.de>

> On Sat, 24 Jan 2009, Miklos Vajna wrote:
>> I'm not familiar with the HTTP code, either, but here is the call stack
>> I see:
>>
>> - builtin-clone calls transport_get_remote_refs()
>> - that will call transport->get_refs_list()
>> - that will call get_refs_via_curl()
>> - that die()s on error, does not use return error()

Which means all protocols actually die?

> Only after writing my email (and just before sending) did I remember that
> I have Mike's HTTP cleanups applied in my Git checkout.  So I was
> analyzing the wrong code.

Hmmm, can't blame you considering it was around 4am ;).

-- 
Cheers,

Sverre Rabbelier