diff mbox

[PATCH/next,3/3] package/dovecot: Add optional support for icu

Message ID 1432194056-10250-3-git-send-email-bernd.kuhls@t-online.de
State Accepted
Headers show

Commit Message

Bernd Kuhls May 21, 2015, 7:40 a.m. UTC
Optional dependency added to fts plugin since Dovecot 2.2.17:
http://hg.dovecot.org/dovecot-2.2/diff/b179bbd226e5/configure.ac

Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
---
 package/dovecot/dovecot.mk |    7 +++++++
 1 file changed, 7 insertions(+)

Comments

Thomas Petazzoni July 13, 2015, 5:04 p.m. UTC | #1
Dear Bernd Kuhls,

On Thu, 21 May 2015 09:40:56 +0200, Bernd Kuhls wrote:
> Optional dependency added to fts plugin since Dovecot 2.2.17:
> http://hg.dovecot.org/dovecot-2.2/diff/b179bbd226e5/configure.ac
> 
> Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
> ---
>  package/dovecot/dovecot.mk |    7 +++++++
>  1 file changed, 7 insertions(+)

Applied, thanks.

It's a bit weird to have it handled as an automatic optional dependency
when several other dovecot dependencies have an explicit sub-option
(zlib, bzip2, etc.). But libcap support was already in this order.

Best regards,

Thomas
diff mbox

Patch

diff --git a/package/dovecot/dovecot.mk b/package/dovecot/dovecot.mk
index 3b4f6d1..58666d0 100644
--- a/package/dovecot/dovecot.mk
+++ b/package/dovecot/dovecot.mk
@@ -40,6 +40,13 @@  else
 DOVECOT_CONF_OPTS += --without-bzlib
 endif
 
+ifeq ($(BR2_PACKAGE_ICU),y)
+DOVECOT_CONF_OPTS += --with-icu
+DOVECOT_DEPENDENCIES += icu
+else
+DOVECOT_CONF_OPTS += --without-icu
+endif
+
 ifeq ($(BR2_PACKAGE_LIBCAP),y)
 DOVECOT_CONF_OPTS += --with-libcap
 DOVECOT_DEPENDENCIES += libcap