On Wed, 27 Feb 2008, Mike Hommey wrote: > 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 proxy url as argument. > > At the same time, we make get_http_walker() take a proxy argument too, and > pass it to http_init(), which makes remote defined proxy be used for more > than get_refs_via_curl(). It's a good idea, but maybe http_init() (and the call chain leading up to it) should take the struct remote, so that it can get anything else you might also have there in the future? For that matter, is it intentional that http-push ignore the proxy setting, which is equally available in http-push, but not used? If so, it should probably get a comment. -Daniel *This .sig left intentionally blank* - 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
