From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Barkalow Subject: Re: maybe breakage with latest git-pull and http protocol Date: Sat, 15 Oct 2005 15:41:35 -0400 (EDT) Message-ID: References: <867jciz18w.fsf@blue.stonehenge.com> <864q7kqsa4.fsf@blue.stonehenge.com> <7vhdbkt8ad.fsf@assigned-by-dhcp.cox.net> <86achcoyvz.fsf@blue.stonehenge.com> <863bn3lypn.fsf@blue.stonehenge.com> <7vd5m6py11.fsf@assigned-by-dhcp.cox.net> <7v1x2mpx6m.fsf@assigned-by-dhcp.cox.net> Mime-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Cc: Nick Hengeveld , "Randal L. Schwartz" , git@vger.kernel.org X-From: git-owner@vger.kernel.org Sat Oct 15 21:38:07 2005 Return-path: Received: from vger.kernel.org ([209.132.176.167]) by ciao.gmane.org with esmtp (Exim 4.43) id 1EQrpd-0003ud-Sh for gcvg-git@gmane.org; Sat, 15 Oct 2005 21:36:38 +0200 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751209AbVJOTgf (ORCPT ); Sat, 15 Oct 2005 15:36:35 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1751211AbVJOTgf (ORCPT ); Sat, 15 Oct 2005 15:36:35 -0400 Received: from iabervon.org ([66.92.72.58]:787 "EHLO iabervon.org") by vger.kernel.org with ESMTP id S1751209AbVJOTge (ORCPT ); Sat, 15 Oct 2005 15:36:34 -0400 Received: (qmail 6562 invoked by uid 1000); 15 Oct 2005 15:41:35 -0400 Received: from localhost (sendmail-bs@127.0.0.1) by localhost with SMTP; 15 Oct 2005 15:41:35 -0400 To: Junio C Hamano In-Reply-To: <7v1x2mpx6m.fsf@assigned-by-dhcp.cox.net> Sender: git-owner@vger.kernel.org Precedence: bulk X-Mailing-List: git@vger.kernel.org Archived-At: On Sat, 15 Oct 2005, Junio C Hamano wrote: > Junio C Hamano writes: > > > merlyn@stonehenge.com (Randal L. Schwartz) writes: > > > >> OK, it happened this morning. While syncing to update from > >> yesterday's version, I got: > > > > Thanks. > > > >> localhost:~/MIRROR/git-GIT % git-pull > >> Fetching refs/heads/master from http://www.kernel.org/pub/scm/git/git.git using http > >> Getting alternates list > >> got 4546738b58a0134eef154231b07d60fc174d56e3 > >> walk 4546738b58a0134eef154231b07d60fc174d56e3 > >> got d402d5566fdf226697a386dfb9858e5d954e9b91 > >> got 873d8e5652c06c3891278f33546c437efc209c2d > >> walk d402d5566fdf226697a386dfb9858e5d954e9b91 > >> error: > >> Getting pack list > >> got 0207ab18a3876249a928e7539d8f594a4f6921f1 > > More interesting is this "error:" without error message. > "Getting pack list" is a signal that we fell back to > fetch_pack(), so this is coming from fetch_object(). > > I see this line could emit an empty error message, if errorstr > is empty. > > if (request->curl_result != CURLE_OK && request->http_code != 416) { > ret = error("%s", request->errorstr); > release_request(request); > return ret; > } > > So if that is the case maybe my previous speculation that we > sometimes forget to issue a necessary request was wrong. We > asked for that object and got an error from cURL library... It looks like we didn't get an error from the cURL library, actually, or it would have printed something. My guess is that it is getting to the point about while the request is still in progress, but I'm not seeing how that could happen. > BTW, I do not think this is related to git.git repository > problem, but I wonder why we do not do fetch_object() against > each altbase in http-fetch.c::fetch(); nobody said you cannot > borrow unpacked object from your neighbour. I believe the code is doing that, but elsewhere. -Daniel *This .sig left intentionally blank*