git/http.h, branch v1.6.2

Work around gcc warnings from curl headers

2008-07-04T07:37:40Z

After master.k.org upgrade, I started seeing these warning messages: transport.c: In function 'get_refs_via_curl': transport.c:458: error: call to '_curl_easy_setopt_err_write_callback' declared with attribute warning: curl_easy_setopt expects a curl_write_callback argument for this option It appears that the curl header wants to enforce the function signature for callback function given to curl_easy_setopt() to be compatible with that of (*curl_write_callback) or fwrite. This patch seems to work the issue around. Signed-off-by: Junio C Hamano

Make walker.fetch_ref() take a struct ref.

2008-04-27T00:36:17Z

This simplifies a few things, makes a few things slightly more complicated, but, more importantly, allows that, when struct ref can represent a symref, http_fetch_ref() can return one. Incidentally makes the string that http_fetch_ref() gets include "refs/" (if appropriate), because that's how the name field of struct ref works. As far as I can tell, the usage in walker:interpret_target() wouldn't have worked previously, if it ever would have been used, which it wouldn't (since the fetch process uses the hash instead of the name of the ref there). Signed-off-by: Daniel Barkalow Signed-off-by: Junio C Hamano

Set proxy override with http_init()

2008-02-27T23:37:57Z

In transport.c, proxy setting (the one from the remote conf) was set through curl_easy_setopt() call, while http.c already does the same with the http.proxy setting. We now just use this infrastructure instead, and make http_init() now take the struct remote as argument so that it can take the http_proxy setting from there, and any other property that would be added later. At the same time, we make get_http_walker() take a struct remote argument too, and pass it to http_init(), which makes remote defined proxy be used for more than get_refs_via_curl(). We leave out http-fetch and http-push, which don't use remotes for the moment, purposefully. Signed-off-by: Mike Hommey Acked-by: Daniel Barkalow Signed-off-by: Junio C Hamano

Clarify that http-push being temporarily disabled with older cURL

2008-01-22T08:48:29Z

Signed-off-by: Junio C Hamano

Move fetch_ref from http-push.c and http-walker.c to http.c

2007-12-15T05:31:59Z

Make the necessary changes to be ok with their difference, and rename the function http_fetch_ref. Signed-off-by: Mike Hommey Signed-off-by: Junio C Hamano

Use strbuf in http code

2007-12-15T05:31:59Z

Also, replace whitespaces with tabs in some places Signed-off-by: Mike Hommey Signed-off-by: Junio C Hamano

Avoid redundant declaration of missing_target()

2007-12-15T05:31:59Z

Signed-off-by: Mike Hommey Signed-off-by: Junio C Hamano

Cleanup variables in http.[ch]

2007-12-09T20:18:42Z

Quite some variables defined as extern in http.h are only used in http.c, and some others, only defined in http.c, were not static. Signed-off-by: Mike Hommey Signed-off-by: Junio C Hamano

Make function to refill http queue a callback

2007-09-19T10:22:30Z

This eliminates the last function provided by the code using http.h as a global symbol, so it should be possible to have multiple programs using http.h in the same executable, and it also adds an argument to that callback, so that info can be passed into the callback without being global. Signed-off-by: Daniel Barkalow Signed-off-by: Junio C Hamano

Refactor http.h USE_CURL_MULTI fill_active_slots().

2007-09-19T10:22:29Z

This removes all of the boilerplate and http-internal stuff from fill_active_slots() and makes it easy to turn into a callback. Signed-off-by: Daniel Barkalow Signed-off-by: Junio C Hamano