Commit e32c29a0 authored by Thomas Petazzoni's avatar Thomas Petazzoni Committed by Peter Korsgaard
Browse files

libcurl: re-enable on non-MMU platforms 

In 9229b82d ('libcurl: needs MMU'),
the libcurl package was disabled on non-MMU systems, due to the usage
of the fork() function in the library.

However, a deeper inspection reveals that fork() is only used in the
implementation of NTLM, an obscure, undocumented, Microsoft specific
authentication method that apparently isn't common anymore. See
http://curl.haxx.se/docs/manpage.html#--ntlm

.

Therefore, this commit re-enables libcurl on non-MMU systems by
explicitly disabling the NTLM support. If someone ever needs NTLM
support in Buildroot's libcurl package, it will always be time to add
a libcurl sub-option to enable it.

Signed-off-by: default avatarThomas Petazzoni <thomas.petazzoni@free-electrons.com>
Signed-off-by: default avatarPeter Korsgaard <jacmet@sunsite.dk>
parent a32a55d8

package/libcurl/Config.in

+0 −1
Original line number Diff line number Diff line 
config BR2_PACKAGE_LIBCURL

	bool "libcurl"

	depends on BR2_USE_MMU # Uses fork()

	help

	  cURL is a tool for getting files from FTP, HTTP, Gopher, Telnet,

	  and Dict servers, using any of the supported protocols.

package/libcurl/libcurl.mk

+7 −1
Original line number Diff line number Diff line
@@ -10,7 +10,13 @@ LIBCURL_SITE = http://curl.haxx.se/download 
LIBCURL_LICENSE = ICS

LIBCURL_LICENSE_FILES = COPYING

LIBCURL_INSTALL_STAGING = YES

LIBCURL_CONF_OPT = --disable-verbose --disable-manual --enable-hidden-symbols



# We disable NTLM support because it uses fork(), which doesn't work

# on non-MMU platforms. Moreover, this authentication method is

# probably almost never used. See

# http://curl.haxx.se/docs/manpage.html#--ntlm.

LIBCURL_CONF_OPT = --disable-verbose --disable-manual \

	--enable-hidden-symbols --disable-ntlm-wb

LIBCURL_CONFIG_SCRIPTS = curl-config



ifeq ($(BR2_PACKAGE_OPENSSL),y)