diff options
| author | Daniel Stenberg <daniel@haxx.se> | 2004-11-08 07:47:08 +0000 |
|---|---|---|
| committer | Daniel Stenberg <daniel@haxx.se> | 2004-11-08 07:47:08 +0000 |
| commit | c688166066d2d34b32ce86b58c03a81fc7e1e190 (patch) | |
| tree | 537c9e93f69f475e3f93c3cfa2c13936d0954c1f | |
| parent | 5dcd2710d4198ae4ec196805fb7d23401ba0de83 (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.ac | 2 | ||||
| -rw-r--r-- | lib/setup.h | 9 |
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 */ |