Merge branch 'db/make-with-curl'
authorJunio C Hamano <gitster@pobox.com>
Thu, 24 Apr 2014 19:31:27 +0000 (12:31 -0700)
committerJunio C Hamano <gitster@pobox.com>
Thu, 24 Apr 2014 19:31:27 +0000 (12:31 -0700)
Ask curl-config how to link with the curl library, instead of
having only a limited configurability knobs in the Makefile.

* db/make-with-curl:
Makefile: allow static linking against libcurl
Makefile: use curl-config to determine curl flags

1  2 
Makefile
diff --combined Makefile
index a53f3a8326c2e62dc79bae7169d64137ac3dab20,e4c93f68ec87a8907a8e80f1043d675328f712e4..74a929b6d40557f9433e8874c97739d2a06cc858
+++ b/Makefile
@@@ -34,8 -34,15 +34,15 @@@ all:
  # git-http-push are not built, and you cannot use http:// and https://
  # transports (neither smart nor dumb).
  #
+ # Define CURL_CONFIG to the path to a curl-config binary other than the
+ # default 'curl-config'.
+ #
+ # Define CURL_STATIC to statically link libcurl.  Only applies if
+ # CURL_CONFIG is used.
+ #
  # Define CURLDIR=/foo/bar if your curl header and library files are in
- # /foo/bar/include and /foo/bar/lib directories.
+ # /foo/bar/include and /foo/bar/lib directories.  This overrides CURL_CONFIG,
+ # but is less robust.
  #
  # Define NO_EXPAT if you do not have expat installed.  git-http-push is
  # not built, and you cannot push using http:// and https:// transports (dumb).
  #
  # Define NEEDS_SSL_WITH_CRYPTO if you need -lssl when using -lcrypto (Darwin).
  #
- # Define NEEDS_SSL_WITH_CURL if you need -lssl with -lcurl (Minix).
+ # Define NEEDS_SSL_WITH_CURL if you need -lssl with -lcurl (Minix).  Only used
+ # if CURLDIR is set.
  #
- # Define NEEDS_IDN_WITH_CURL if you need -lidn when using -lcurl (Minix).
+ # Define NEEDS_IDN_WITH_CURL if you need -lidn when using -lcurl (Minix).  Only
+ # used if CURLDIR is set.
  #
  # Define NEEDS_LIBICONV if linking with libc is not enough (Darwin).
  #
                # Try "-Wl,-rpath=$(CURLDIR)/$(lib)" in such a case.
                BASIC_CFLAGS += -I$(CURLDIR)/include
                CURL_LIBCURL = -L$(CURLDIR)/$(lib) $(CC_LD_DYNPATH)$(CURLDIR)/$(lib) -lcurl
+               ifdef NEEDS_SSL_WITH_CURL
+                       CURL_LIBCURL += -lssl
+                       ifdef NEEDS_CRYPTO_WITH_SSL
+                               CURL_LIBCURL += -lcrypto
+                       endif
+               endif
+               ifdef NEEDS_IDN_WITH_CURL
+                       CURL_LIBCURL += -lidn
+               endif
        else
-               CURL_LIBCURL = -lcurl
-       endif
-       ifdef NEEDS_SSL_WITH_CURL
-               CURL_LIBCURL += -lssl
-               ifdef NEEDS_CRYPTO_WITH_SSL
-                       CURL_LIBCURL += -lcrypto
+               CURL_CONFIG ?= curl-config
+               BASIC_CFLAGS += $(shell $(CURL_CONFIG) --cflags)
+               ifdef CURL_STATIC
+                       CURL_LIBCURL = $(shell $(CURL_CONFIG) --static-libs)
+                       ifeq "$(CURL_LIBCURL)" ""
+                               $(error libcurl not detected or not compiled with static support)
+                       endif
+               else
+                       CURL_LIBCURL = $(shell $(CURL_CONFIG) --libs)
+                       ifeq "$(CURL_LIBCURL)" ""
+                               $(error libcurl not detected; try setting CURLDIR)
+                       endif
                endif
        endif
-       ifdef NEEDS_IDN_WITH_CURL
-               CURL_LIBCURL += -lidn
-       endif
  
        REMOTE_CURL_PRIMARY = git-remote-http$X
        REMOTE_CURL_ALIASES = git-remote-https$X git-remote-ftp$X git-remote-ftps$X
@@@ -2102,7 -2123,7 +2123,7 @@@ pdf
  
  XGETTEXT_FLAGS = \
        --force-po \
 -      --add-comments \
 +      --add-comments=TRANSLATORS: \
        --msgid-bugs-address="Git Mailing List <git@vger.kernel.org>" \
        --from-code=UTF-8
  XGETTEXT_FLAGS_C = $(XGETTEXT_FLAGS) --language=C \