aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDaniel Stenberg <daniel@haxx.se>2004-11-08 07:47:08 +0000
committerDaniel Stenberg <daniel@haxx.se>2004-11-08 07:47:08 +0000
commitc688166066d2d34b32ce86b58c03a81fc7e1e190 (patch)
tree537c9e93f69f475e3f93c3cfa2c13936d0954c1f
parent5dcd2710d4198ae4ec196805fb7d23401ba0de83 (diff)
check for and require tld.h to be present before libidn usage is activated
in the build, since libidn 0.3.X didn't have the header and we don't support that old libidn versions anyway. This was mentioned on the list by Jean-Philippe Barrette-LaPierre and in bug report #1062264.
-rw-r--r--configure.ac2
-rw-r--r--lib/setup.h9
2 files changed, 6 insertions, 5 deletions
diff --git a/configure.ac b/configure.ac
index 3a01b2a45..091c869b8 100644
--- a/configure.ac
+++ b/configure.ac
@@ -984,7 +984,7 @@ case "$LIBIDN" in
curl_idn_msg="enabled"
dnl different versions of libidn have different setups of these:
AC_CHECK_FUNCS( idn_free idna_strerror tld_strerror)
- AC_CHECK_HEADERS( idn-free.h )
+ AC_CHECK_HEADERS( idn-free.h tld.h )
fi
;;
diff --git a/lib/setup.h b/lib/setup.h
index f121b7320..e535279c8 100644
--- a/lib/setup.h
+++ b/lib/setup.h
@@ -281,11 +281,12 @@ typedef int curl_socket_t;
#undef HAVE_ALARM
#endif
-#ifdef HAVE_LIBIDN
-/* This could benefit from additional checks that some of the used/important
- header files are present as well before we define the USE_* define. */
+#if defined(HAVE_LIBIDN) && defined(HAVE_TLD_H)
+/* The lib was present and the tld.h header (which is missing in libidn 0.3.X
+ but we only work with libidn 0.4.1 or later) */
#define USE_LIBIDN
-#define LIBIDN_REQUIRED_VERSION "0.4.1"
#endif
+#define LIBIDN_REQUIRED_VERSION "0.4.1"
+
#endif /* __CONFIG_H */