Message ID | 1429210360-4753-1-git-send-email-bernd.kuhls@t-online.de |
---|---|
State | Rejected |
Headers | show |
Bernd, All, On 2015-04-16 20:52 +0200, Bernd Kuhls spake thusly: > > Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de> > --- > package/exim/exim.mk | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/package/exim/exim.mk b/package/exim/exim.mk > index 67dbd64..a95559d 100644 > --- a/package/exim/exim.mk > +++ b/package/exim/exim.mk > @@ -9,7 +9,7 @@ EXIM_SOURCE = exim-$(EXIM_VERSION).tar.bz2 > EXIM_SITE = ftp://ftp.exim.org/pub/exim/exim4 > EXIM_LICENSE = GPLv2+ > EXIM_LICENSE_FILES = LICENCE > -EXIM_DEPENDENCIES = pcre berkeleydb host-pkgconf > +EXIM_DEPENDENCIES = pcre berkeleydb > > # Modify a variable value. It must already exist in the file, either > # commented or not. > @@ -63,7 +63,7 @@ endef > endif > > ifeq ($(BR2_PACKAGE_OPENSSL),y) > -EXIM_DEPENDENCIES += openssl > +EXIM_DEPENDENCIES += openssl host-pkgconf Well, even though we currently only support berkleydb as a DB backend, exim can also use sqlite. And detecting sqlite is done via pkg-config. Also, exim uses pkg-config for a lot of other stuff; quoting their (awfull) buildsystem: for name_mod in \ CDB DBM:dbmdb DNSDB DSEARCH IBASE LSEARCH MYSQL NIS NISPLUS ORACLE \ PASSWD PGSQL SQLITE TESTDB WHOSON do emit_module_rule $name_mod done and emit_module_rule() contains this code: [--SNIP--] echo "LOOKUP_${mod_name}_INCLUDE = $(pkg-config --cflags $pkgconf)" echo "LOOKUP_${mod_name}_LIBS = $(pkg-config --libs $pkgconf)" [--SNIP--] So, even though we do not really need it when OpenSSL is disabled, I think we want to keep it as an uncoditionaly dependency. Regards, Yann E. MORIN. > define EXIM_USE_DEFAULT_CONFIG_FILE_OPENSSL > $(call exim-config-change,SUPPORT_TLS,yes) > $(call exim-config-change,USE_OPENSSL_PC,openssl) > -- > 1.7.10.4 > > _______________________________________________ > buildroot mailing list > buildroot@busybox.net > http://lists.busybox.net/mailman/listinfo/buildroot
Dear Bernd Kuhls, On Thu, 16 Apr 2015 20:52:40 +0200, Bernd Kuhls wrote: > > Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de> > --- > package/exim/exim.mk | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) Following the feedback from Yann, who suggested to keep the dependency unconditional, I marked your patch as Rejected in patchwork. Thanks! Thomas
diff --git a/package/exim/exim.mk b/package/exim/exim.mk index 67dbd64..a95559d 100644 --- a/package/exim/exim.mk +++ b/package/exim/exim.mk @@ -9,7 +9,7 @@ EXIM_SOURCE = exim-$(EXIM_VERSION).tar.bz2 EXIM_SITE = ftp://ftp.exim.org/pub/exim/exim4 EXIM_LICENSE = GPLv2+ EXIM_LICENSE_FILES = LICENCE -EXIM_DEPENDENCIES = pcre berkeleydb host-pkgconf +EXIM_DEPENDENCIES = pcre berkeleydb # Modify a variable value. It must already exist in the file, either # commented or not. @@ -63,7 +63,7 @@ endef endif ifeq ($(BR2_PACKAGE_OPENSSL),y) -EXIM_DEPENDENCIES += openssl +EXIM_DEPENDENCIES += openssl host-pkgconf define EXIM_USE_DEFAULT_CONFIG_FILE_OPENSSL $(call exim-config-change,SUPPORT_TLS,yes) $(call exim-config-change,USE_OPENSSL_PC,openssl)
Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de> --- package/exim/exim.mk | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)