Tarmigan <tarmigan+git@xxxxxxxxx> wrote: > On Sun, Nov 8, 2009 at 9:18 PM, Junio C Hamano <gitster@xxxxxxxxx> wrote: > > * sp/smart-http (2009-11-04) 30 commits > > + test smart http fetch and push > > I am trying to test smart http, and have had to set > DEFAULT_HTTPD_PATH='/usr/sbin/httpd' > DEFAULT_HTTPD_MODULE_PATH='/usr/lib64/httpd/modules' on Centos. > Perhaps this failing test is just a difference in the default Apache > and curl configurations. *sigh* > --- exp 2009-11-09 07:33:19.000000000 +0000 > +++ act 2009-11-09 07:33:19.000000000 +0000 > @@ -6,15 +6,16 @@ > < Pragma: no-cache > < Cache-Control: no-cache, max-age=0, must-revalidate > < Content-Type: application/x-git-upload-pack-advertisement > -< > > POST /smart/repo.git/git-upload-pack HTTP/1.1 > +> Accept: */* > > Accept-Encoding: deflate, gzip > > Content-Type: application/x-git-upload-pack-request > > Accept: application/x-git-upload-pack-response The first delta is a missing blank line between requests. I probably can work around that with some sort of sed hackery and its likely caused by a different libcurl version. The second delta is libcurl sending *two* Accept headers to the server. The first was set by the library as "*/*", but the second was set by git-remote-curl. Here your libcurl is just plain *wrong*. The application has said "I only support one content type, this one" and the library still went and told the server "I accept anything you have, just give it to me!". I'm not sure what to do about this breakage in curl. We may have to relax how we use HTTP since at least one version of libcurl can't get this minor detail right. > > Content-Length: xxx > > +> 0073want 1937bb05802e1973cc2e437c13e9f1845941b785 > multi_ack_detailed side-band-64k thin-pack no-progress ofs-delta > +> 00000009done Hmmph. Your libcurl also included the POST body in the verbose message. -- Shawn. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html