[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH] configure: fix curl installed in non-standard place
From: |
Nathan Froyd |
Subject: |
[Qemu-devel] [PATCH] configure: fix curl installed in non-standard places |
Date: |
Tue, 21 Jul 2009 07:15:06 -0700 |
configure helpfully uses curl-config to figure out appropriate -L and -l
options for the compiler, but fails to check for any necessary -I
options. If, perchance, you are using a compiler whose #include paths
have an older version of curl.h lying about, but have a newer version of
curl installed locally (say, $HOME), the small test program in configure
will succeed (with a warning, possibly). But the compilation of
block/curl.c will fail because curl.c requires more up-to-date features.
To avoid this, grab --cflags from curl-config too. It's not completely
foolproof (the test in configure should really be checking for all the
features that block/curl.c uses or block/curl.c should gracefully
cooperate with older versions), but it's more correct than what we have.
---
Makefile | 1 +
configure | 4 +++-
2 files changed, 4 insertions(+), 1 deletions(-)
diff --git a/Makefile b/Makefile
index dc95869..9899651 100644
--- a/Makefile
+++ b/Makefile
@@ -175,6 +175,7 @@ LIBS+=$(VDE_LIBS)
obj-$(CONFIG_XEN) += xen_backend.o xen_devconfig.o
obj-$(CONFIG_XEN) += xen_console.o xenfb.o xen_disk.o xen_nic.o
+CPPFLAGS+=$(CURL_INCLUDES)
LIBS+=$(CURL_LIBS)
cocoa.o: cocoa.m
diff --git a/configure b/configure
index 0db885b..0f60328 100755
--- a/configure
+++ b/configure
@@ -1126,8 +1126,9 @@ if test "$curl" = "yes" ; then
#include <curl/curl.h>
int main(void) { return curl_easy_init(); }
EOF
+ curl_includes=`curl-config --cflags 2>/dev/null`
curl_libs=`curl-config --libs 2>/dev/null`
- if $cc $ARCH_CFLAGS $curl_libs -o $TMPE $TMPC > /dev/null 2> /dev/null ; then
+ if $cc $ARCH_CFLAGS $curl_includes $curl_libs -o $TMPE $TMPC > /dev/null 2>
/dev/null ; then
curl=yes
fi
fi # test "$curl"
@@ -1720,6 +1721,7 @@ if test "$inotify" = "yes" ; then
fi
if test "$curl" = "yes" ; then
echo "CONFIG_CURL=y" >> $config_host_mak
+ echo "CURL_INCLUDES=$curl_includes" >> $config_host_mak
echo "CURL_LIBS=$curl_libs" >> $config_host_mak
echo "#define CONFIG_CURL 1" >> $config_host_h
fi
--
1.6.2.4
- [Qemu-devel] [PATCH] configure: fix curl installed in non-standard places,
Nathan Froyd <=