2 messages, 2 authors, 2016-06-15 · open the first message on its own page
From: Junio C Hamano <hidden>
Date: 2016-06-15 22:46:56
Mark Lodato [off-list ref] writes:
If this patch series is accepted, I will make a cleaner version that includes this change.
Sorry, but I do not understand this part of your message.
From: Mark Lodato <hidden>
Date: 2016-06-15 22:46:56
On Fri, Jun 12, 2009 at 8:14 PM, Junio C Hamano[off-list ref] wrote:
Mark Lodato [off-list ref] writes:quoted
If this patch series is accepted, I will make a cleaner version that includes this change.Sorry, but I do not understand this part of your message.
Sorry about that. I meant that I have cleaned up the code as you suggested (see diff below), and that if you decide to include the patch series into git.git (I see now you included it in pu), I can either submit an additional patch to perform the cleanup, or submit a new "v2" patch series incorporating these changes. Is one preferred over the other? Also, I wasn't sure where to put the #defines; I chose to put them in http.h, but should they go in http.c? Thanks for the feedback! Mark
diff --git c/http.c i/http.c
index 6ae59b6..7659ef4 100644
--- c/http.c
+++ i/http.c@@ -213,16 +213,8 @@ static CURL *get_curl_handle(void) if (ssl_cert != NULL) curl_easy_setopt(result, CURLOPT_SSLCERT, ssl_cert); if (has_cert_password()) - curl_easy_setopt(result, -#if LIBCURL_VERSION_NUM >= 0x071700 - CURLOPT_KEYPASSWD, -#elif LIBCURL_VERSION_NUM >= 0x070903 - CURLOPT_SSLKEYPASSWD, -#else - CURLOPT_SSLCERTPASSWD, -#endif - ssl_cert_password); -#if LIBCURL_VERSION_NUM >= 0x070902 + curl_easy_setopt(result, CURLOPT_KEYPASSWD, ssl_cert_password); +#ifndef NO_CURLOPT_SSLKEY if (ssl_key != NULL) curl_easy_setopt(result, CURLOPT_SSLKEY, ssl_key); #endif
diff --git c/http.h i/http.h
index 26abebe..b49c280 100644
--- c/http.h
+++ i/http.h@@ -29,6 +29,12 @@ #define curl_global_init(a) do { /* nothing */ } while(0) #endif +#if LIBCURL_VERSION_NUM < 0x070903 +#define CURLOPT_KEYPASSWD CURLOPT_SSLCERTPASSWD +#elif LIBCURL_VERSION_NUM < 0x071700 +#define CURLOPT_KEYPASSWD CURLOPT_SSLKEYPASSWD +#endif + #if (LIBCURL_VERSION_NUM < 0x070c04) || (LIBCURL_VERSION_NUM == 0x071000) #define NO_CURL_EASY_DUPHANDLE #endif