mutt: Bump version to 1.5.23

PATCH Superseded Bernd Kuhls 2 participants 3 messages Apr 13, 2014

lore

Share & tools

lore

Bernd Kuhls

author

2014-04-13 09:36 UTC #0 lore raw

PATCH 1/2 mutt: Bump version to 1.5.23

removed upstream applied patch mutt-02-automake.patch
http://dev.mutt.org/trac/changeset/0488deb39a35

Signed-off-by Bernd Kuhls <[email protected]>

---
 package/mutt/mutt-02-automake.patch |   16 ----------------
 package/mutt/mutt.mk                |    4 ++--
 2 files changed, 2 insertions(+), 18 deletions(-)
 delete mode 100644 package/mutt/mutt-02-automake.patch

package/mutt/mutt-02-automake.patch deleted +0 ~~-16~~

deleted file mode 100644
index 19950a9..0000000
@@ -1,16 +0,0 @@
-Fix for automake 1.12+
-http://dev.mutt.org/trac/ticket/3579
-
-Signed-off-by: Gustavo Zacarias <[email protected]>
-
-diff -Nura mutt-1.5.21.orig/configure.ac mutt-1.5.21/configure.ac
---- mutt-1.5.21.orig/configure.ac	2013-09-16 11:07:25.699781892 -0300
-+++ mutt-1.5.21/configure.ac	2013-09-16 11:07:45.120419960 -0300
-@@ -27,7 +27,6 @@
- 
- AC_PROG_CC
- AC_ISC_POSIX
--AM_C_PROTOTYPES
- if test "x$U" != "x"; then
-   AC_MSG_ERROR(Compiler not ANSI compliant)
- fi

package/mutt/mutt.mk +2 -2

index 0c17589..5aa18eb 100644
@@ -4,8 +4,8 @@
 #
 ################################################################################
 
-MUTT_VERSION = 1.5.21
-MUTT_SITE = http://downloads.sourceforge.net/project/mutt/mutt-dev
+MUTT_VERSION = 1.5.23
+MUTT_SITE = http://downloads.sourceforge.net/project/mutt/mutt
 MUTT_LICENSE = GPLv2+
 MUTT_LICENSE_FILES = GPL
 MUTT_DEPENDENCIES = ncurses

Signature · 1 line

1.7.10.4

Bernd Kuhls

author

2014-04-13 09:36 UTC #1 lore raw

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

Signed-off-by Bernd Kuhls <[email protected]>

---
 package/mutt/Config.in |   16 +++++++++++++++-
 package/mutt/mutt.mk   |   33 ++++++++++++++++++++++++++++++++-
 2 files changed, 47 insertions(+), 2 deletions(-)

package/mutt/Config.in +15 -1

index 430eeb8..372c137 100644
@@ -1,4 +1,4 @@
-config BR2_PACKAGE_MUTT
+menuconfig BR2_PACKAGE_MUTT
 	bool "mutt"
 	depends on BR2_USE_WCHAR
 	depends on BR2_USE_MMU # fork()

@@ -8,6 +8,20 @@ config BR2_PACKAGE_MUTT
 
 	  http://www.mutt.org/
 
+if BR2_PACKAGE_MUTT
+
+config BR2_PACKAGE_MUTT_IMAP
+	bool "imap"
+	help
+	  IMAP support
+
+config BR2_PACKAGE_MUTT_POP3
+	bool "pop3"
+	help
+	  POP3 support
+
+endif # BR2_PACKAGE_MUTT
+
 comment "mutt needs a toolchain w/ wchar"
 	depends on BR2_USE_MMU
 	depends on !BR2_USE_WCHAR

package/mutt/mutt.mk +32 -1

index 5aa18eb..0644276 100644
@@ -9,7 +9,38 @@ MUTT_SITE = http://downloads.sourceforge.net/project/mutt/mutt
 MUTT_LICENSE = GPLv2+
 MUTT_LICENSE_FILES = GPL
 MUTT_DEPENDENCIES = ncurses
-MUTT_CONF_OPT = --disable-iconv --disable-smtp
+MUTT_CONF_OPT = --disable-smtp
 MUTT_AUTORECONF = YES
 
+ifeq ($(BR2_PACKAGE_LIBICONV),y)
+MUTT_DEPENDENCIES += libiconv
+MUTT_CONF_OPT += --enable-iconv
+else
+MUTT_CONF_OPT += --disable-iconv
+endif
+
+ifeq ($(BR2_PACKAGE_MUTT_IMAP),y)
+MUTT_CONF_OPT += --enable-imap
+else
+MUTT_CONF_OPT += --disable-imap
+endif
+
+ifeq ($(BR2_PACKAGE_MUTT_POP3),y)
+MUTT_CONF_OPT += --enable-pop
+else
+MUTT_CONF_OPT += --disable-pop
+endif
+
+# SSL support is only used by imap or pop3 module
+ifeq ($(findstring xy,x$(BR2_PACKAGE_MUTT_IMAP)$(BR2_PACKAGE_MUTT_POP3)),xy)
+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))

Signature · 1 line

1.7.10.4

Thomas Petazzoni

2014-04-13 12:00 UTC #2 lore raw

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

Dear Bernd Kuhls,

package/mutt/mutt.mk MUTT_SITE = http://downloads.sourceforge.net/project/mutt/mutt · line 20

Quoted 6 lines · Bernd Kuhls

package/mutt/mutt.mk +6 -0

+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.

package/mutt/mutt.mk MUTT_SITE = http://downloads.sourceforge.net/project/mutt/mutt · line 35

Quoted 2 lines

package/mutt/mutt.mk +2 -0

+# 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),)

package/mutt/mutt.mk MUTT_SITE = http://downloads.sourceforge.net/project/mutt/mutt · line 46

Quoted 11 lines

package/mutt/mutt.mk +10 -0

+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

Signature · 3 lines

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