[17/49,v2] package/asterisk: add optional openldap dependency

Message ID 81e461eac586fd2aab201f5b3d2c0600ee0304e7.1504993178.git.yann.morin.1998@free.fr
State Accepted
Headers show
Series
  • [01/49,v2] package/asterisk: new package
Related show

Commit Message

Yann E. MORIN Sept. 9, 2017, 9:39 p.m.
Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
---
 package/asterisk/asterisk.mk | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

Comments

Bernd Kuhls Sept. 14, 2017, 2:36 p.m. | #1
Hi Yann,

Am Sat, 09 Sep 2017 23:39:23 +0200 schrieb Yann E. MORIN:

> +ifeq ($(BR2_PACKAGE_OPENLDAP),y)
> +ASTERISK_DEPENDENCIES += openldap
> +ASTERISK_CONF_OPTS += --without-ldap

this should be "--with-ldap"

> +else
> +ASTERISK_CONF_OPTS += --without-ldap
> +endif

Regards, Bernd
Yann E. MORIN Sept. 16, 2017, 4:46 p.m. | #2
Bernd, All,

On 2017-09-14 16:36 +0200, Bernd Kuhls spake thusly:
> Hi Yann,
> 
> Am Sat, 09 Sep 2017 23:39:23 +0200 schrieb Yann E. MORIN:
> 
> > +ifeq ($(BR2_PACKAGE_OPENLDAP),y)
> > +ASTERISK_DEPENDENCIES += openldap
> > +ASTERISK_CONF_OPTS += --without-ldap
> 
> this should be "--with-ldap"

Uhu... Good catch. ;-)

Regards,
Yann E. MORIN.

> > +else
> > +ASTERISK_CONF_OPTS += --without-ldap
> > +endif
> 
> Regards, Bernd
> 
> _______________________________________________
> buildroot mailing list
> buildroot@busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot

Patch

diff --git a/package/asterisk/asterisk.mk b/package/asterisk/asterisk.mk
index 279b2b2eb3..e46538db17 100644
--- a/package/asterisk/asterisk.mk
+++ b/package/asterisk/asterisk.mk
@@ -59,7 +59,6 @@  ASTERISK_CONF_OPTS = \
 	--without-jack \
 	--without-uriparser \
 	--without-kqueue \
-	--without-ldap \
 	--without-libedit \
 	--without-libxslt \
 	--without-ltdl \
@@ -154,6 +153,13 @@  else
 ASTERISK_CONF_OPTS += --without-ilbc
 endif
 
+ifeq ($(BR2_PACKAGE_OPENLDAP),y)
+ASTERISK_DEPENDENCIES += openldap
+ASTERISK_CONF_OPTS += --without-ldap
+else
+ASTERISK_CONF_OPTS += --without-ldap
+endif
+
 ifeq ($(BR2_PACKAGE_LIBCURL),y)
 ASTERISK_DEPENDENCIES += libcurl
 ASTERISK_CONF_OPTS += --with-libcurl