Hi, On Sat, Aug 7, 2010 at 11:34 AM, Spencer E. Olson <olsonse@xxxxxxxxx> wrote: >[snip] > @@ -41,6 +41,7 @@ static long curl_low_speed_time = -1; > static int curl_ftp_no_epsv; > static const char *curl_http_proxy; > static char *user_name, *user_pass; > +static const char *user_agent = NULL; This can be skipped, I think. >[snip] > @@ -279,7 +283,10 @@ static CURL *get_curl_handle(void) > if (getenv("GIT_CURL_VERBOSE")) > curl_easy_setopt(result, CURLOPT_VERBOSE, 1); > > - curl_easy_setopt(result, CURLOPT_USERAGENT, GIT_USER_AGENT); > + if (user_agent != NULL) > + curl_easy_setopt(result, CURLOPT_USERAGENT, user_agent); > + else > + curl_easy_setopt(result, CURLOPT_USERAGENT, GIT_USER_AGENT); Hmm, perhaps curl_easy_setopt(result, CURLOPT_USERAGENT, user_agent ? user_agent : GIT_USER_AGENT); to replace the if-else? -- Cheers, Ray Chuan -- 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