[Buildroot] [PATCH 2/2] mutt: Add options for imap/pop3 support, enable optional iconv/OpenSSL support

[Thomas Petazzoni <thomas.petazzoni@free-electrons.com>]

Dear Bernd Kuhls,

On Sun, 13 Apr 2014 11:36:03 +0200, Bernd Kuhls wrote:

> +ifeq ($(BR2_PACKAGE_LIBICONV),y)
> +MUTT_DEPENDENCIES += libiconv
> +MUTT_CONF_OPT += --enable-iconv
> +else
> +MUTT_CONF_OPT += --disable-iconv
> +endif

The libiconv package provides libiconv support for uClibc toolchains
that don't have locale enabled.

But if you're using an uClibc toolchain with locale support enabled, or
a glibc toolchain, then iconv support *is* available, without the
libiconv package enabled. However, with your code, iconv support will
be disabled in mutt in such cases.

Maybe this is an area we need to improve a bit, by providing a Kconfig
option or something like that.

> +# SSL support is only used by imap or pop3 module
> +ifeq ($(findstring xy,x$(BR2_PACKAGE_MUTT_IMAP)$(BR2_PACKAGE_MUTT_POP3)),xy)

This could maybe be:

ifneq ($(BR2_PACKAGET_MUTT_IMAP)$(BR2_PACKAGE_MUTT_POP3),)

> +ifeq ($(BR2_PACKAGE_OPENSSL),y)
> +MUTT_DEPENDENCIES += openssl
> +MUTT_CONF_OPT += --with-ssl=$(STAGING_DIR)/usr
> +else
> +MUTT_CONF_OPT += --without-ssl
> +endif
> +else
> +MUTT_CONF_OPT += --without-ssl
> +endif
> +
>  $(eval $(autotools-package))

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com