Skip to contentSkip to conversation

libcurl: use c-ares if available

PATCH New John Keeping 2 participants 2 messages May 2, 2015 – May 3, 2015

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 libcurl: use c-ares if available

By default libcurl uses the C library's DNS resolver which is synchronous, even if an application is using libcurl's non-blocking mode of operation.

Configure libcurl to use c-ares if it is selected so that it can resolve addresses asynchronously if required.

---
 package/libcurl/libcurl.mk | 5 +++++
 1 file changed, 5 insertions(+)
package/libcurl/libcurl.mk +5 -0
index 014c5e3..825c844 100644
@@ -46,6 +46,11 @@ LIBCURL_CONF_OPTS += --without-ssl --without-gnutls \
 	--without-polarssl --without-nss
 endif
 
+ifeq ($(BR2_PACKAGE_C_ARES),y)
+LIBCURL_DEPENDENCIES += c-ares
+LIBCURL_CONF_OPTS += --enable-ares
+endif
+
 # Configure curl to support libssh2
 ifeq ($(BR2_PACKAGE_LIBSSH2),y)
 LIBCURL_DEPENDENCIES += libssh2
Signature · 1 line
2.4.0.rc2.272.g764ef63
#1 lore raw

PATCH 1/1 libcurl: use c-ares if available

Quoted 20 lines
Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line

"John" == John Keeping <[email protected]> writes:

By default libcurl uses the C library's DNS resolver which is synchronous, even if an application is using libcurl's non-blocking mode of operation.

Configure libcurl to use c-ares if it is selected so that it can resolve addresses asynchronously if required.

---
 package/libcurl/libcurl.mk | 5 +++++
 1 file changed, 5 insertions(+)
package/libcurl/libcurl.mk +0 -0
index 014c5e3..825c844 100644

package/libcurl/libcurl.mk LIBCURL_CONF_OPTS += --without-ssl --without-gnutls \ · line 52

Quoted 7 lines
package/libcurl/libcurl.mk +4 -0
@@ -46,6 +46,11 @@ LIBCURL_CONF_OPTS += --without-ssl --without-gnutls \
 	--without-polarssl --without-nss
 endif
 
+ifeq ($(BR2_PACKAGE_C_ARES),y)
+LIBCURL_DEPENDENCIES += c-ares
+LIBCURL_CONF_OPTS += --enable-ares
+endif

We prefer to be explicit about these dependencies, so I've added a --disable-ares in the else clause and committed, thanks.

Signature · 1 line
Bye, Peter Korsgaard