From: Junio C Hamano Date: Fri, 28 Oct 2016 16:01:13 +0000 (-0700) Subject: Merge branch 'ak/curl-imap-send-explicit-scheme' into maint X-Git-Tag: v2.10.2~21 X-Git-Url: https://git.lorimer.id.au/gitweb.git/diff_plain/f98180a982168d767529c36337b1aadfa0d07788?hp=-c Merge branch 'ak/curl-imap-send-explicit-scheme' into maint When we started cURL to talk to imap server when a new enough version of cURL library is available, we forgot to explicitly add imap(s):// before the destination. To some folks, that didn't work and the library tried to make HTTP(s) requests instead. * ak/curl-imap-send-explicit-scheme: imap-send: Tell cURL to use imap:// or imaps:// --- f98180a982168d767529c36337b1aadfa0d07788 diff --combined imap-send.c index 0f5f4760e9,7dd5acf9a9..adb9738c30 --- a/imap-send.c +++ b/imap-send.c @@@ -511,7 -511,7 +511,7 @@@ static int nfsnprintf(char *buf, int bl va_start(va, fmt); if (blen <= 0 || (unsigned)(ret = vsnprintf(buf, blen, fmt, va)) >= (unsigned)blen) - die("Fatal: buffer too small. Please report a bug."); + die("BUG: buffer too small. Please report a bug."); va_end(va); return ret; } @@@ -1410,6 -1410,7 +1410,7 @@@ static CURL *setup_curl(struct imap_ser curl_easy_setopt(curl, CURLOPT_USERNAME, server.user); curl_easy_setopt(curl, CURLOPT_PASSWORD, server.pass); + strbuf_addstr(&path, server.use_ssl ? "imaps://" : "imap://"); strbuf_addstr(&path, server.host); if (!path.len || path.buf[path.len - 1] != '/') strbuf_addch(&path, '/'); @@@ -1443,7 -1444,6 +1444,7 @@@ if (0 < verbosity || getenv("GIT_CURL_VERBOSE")) curl_easy_setopt(curl, CURLOPT_VERBOSE, 1L); + setup_curl_trace(curl); return curl; } @@@ -1495,12 -1495,16 +1496,12 @@@ static int curl_append_msgs_to_imap(str } #endif -int main(int argc, char **argv) +int cmd_main(int argc, const char **argv) { struct strbuf all_msgs = STRBUF_INIT; int total; int nongit_ok; - git_extract_argv0_path(argv[0]); - - git_setup_gettext(); - setup_git_directory_gently(&nongit_ok); git_imap_config();