7d69a79624
It is often difficult to know exactly when make will expand the variable, and usually it can only be expanded after the dependencies have been built (e.g. pkg-config or the .pc file). Using a backtick instead makes it very clear that it will be expanded only while executing the command. This change is useful for two cases: 1. The per-package staging (and host) directory will be created as part of the configure step, so any $(shell ...) variable that is used in the configure step will fail because the directory doesn't exist yet. 2. 'make printvars' evaluates the variables it prints. It will therefore trigger a lot of errors from missing .pc files and others. The backticks, on the other hand, are not expanded, so with this change the output of 'make printvars' becomes clean again. This commit contains only the easy changes: replace $(shell ...) with `...`, and also replace ' with " where needed. Follow-up commits will tackle the more complicated cases that need additional explanation. After this change, the following instances of $(shell ...) will remain: - All assignments that use := - All variables that are used in make conditionals (which don't expand the backticks). - All variables that only refer to system executables and make variables that don't change. - The calls to check-host-* in dependencies.mk, because it is eval'ed. [Original patch by Fabio Porcedda, but extended quite a bit by Arnout.] Signed-off-by: Fabio Porcedda <fabio.porcedda@gmail.com> Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
73 lines
2.1 KiB
Makefile
73 lines
2.1 KiB
Makefile
################################################################################
|
|
#
|
|
# openldap
|
|
#
|
|
################################################################################
|
|
|
|
OPENLDAP_VERSION = 2.4.41
|
|
OPENLDAP_SOURCE = openldap-$(OPENLDAP_VERSION).tgz
|
|
OPENLDAP_SITE = ftp://ftp.openldap.org/pub/OpenLDAP/openldap-release
|
|
OPENLDAP_LICENSE = OpenLDAP Public License
|
|
OPENLDAP_LICENSE_FILES = LICENSE
|
|
OPENLDAP_INSTALL_STAGING = YES
|
|
OPENLDAP_DEPENDENCIES = host-pkgconf
|
|
|
|
ifeq ($(BR2_PACKAGE_OPENSSL),y)
|
|
OPENLDAP_TLS = openssl
|
|
OPENLDAP_DEPENDENCIES += openssl
|
|
else ifeq ($(BR2_PACKAGE_GNUTLS),y)
|
|
OPENLDAP_TLS = gnutls
|
|
OPENLDAP_DEPENDENCIES += gnutls
|
|
else ifeq ($(BR2_PACKAGE_LIBNSS),y)
|
|
OPENLDAP_TLS = moznss
|
|
OPENLDAP_DEPENDENCIES += libnss
|
|
OPENLDAP_CPPFLAGS = \
|
|
-I$(STAGING_DIR)/usr/include/nss \
|
|
-I$(STAGING_DIR)/usr/include/nspr
|
|
else
|
|
OPENLDAP_TLS = no
|
|
endif
|
|
|
|
ifeq ($(BR2_PACKAGE_OPENSSL),y)
|
|
OPENLDAP_MP = bignum
|
|
OPENLDAP_DEPENDENCIES += openssl
|
|
OPENLDAP_CONF_ENV = LIBS="`$(PKG_CONFIG_HOST_BINARY) --libs libssl libcrypto`"
|
|
else ifeq ($(BR2_PACKAGE_GMP),y)
|
|
OPENLDAP_MP = gmp
|
|
OPENLDAP_DEPENDENCIES += gmp
|
|
else
|
|
OPENLDAP_MP = longlong
|
|
endif
|
|
|
|
OPENLDAP_CONF_ENV += ac_cv_func_memcmp_working=yes
|
|
|
|
OPENLDAP_CONF_OPTS += \
|
|
--enable-syslog \
|
|
--disable-proctitle \
|
|
--disable-slapd \
|
|
--with-yielding-select \
|
|
--sysconfdir=/etc \
|
|
--enable-dynamic=$(if $(BR2_STATIC_LIBS),no,yes) \
|
|
--with-tls=$(OPENLDAP_TLS) \
|
|
--with-mp=$(OPENLDAP_MP) \
|
|
CPPFLAGS="$(TARGET_CPPFLAGS) $(OPENLDAP_CPPFLAGS)"
|
|
|
|
# Somehow, ${STRIP} does not percolates through to the shtool script
|
|
# used to install the executables; thus, that script tries to run the
|
|
# executable it is supposed to install, resulting in an error.
|
|
OPENLDAP_MAKE_ENV = STRIP="$(TARGET_STRIP)"
|
|
|
|
ifeq ($(BR2_PACKAGE_OPENLDAP_CLIENTS),)
|
|
OPENLDAP_CLIENTS = \
|
|
ldapurl ldapexop ldapcompare ldapwhoami \
|
|
ldappasswd ldapmodrdn ldapdelete ldapmodify \
|
|
ldapsearch
|
|
define OPENLDAP_REMOVE_CLIENTS
|
|
$(RM) -f $(foreach p,$(OPENLDAP_CLIENTS),$(TARGET_DIR)/usr/bin/$(p))
|
|
$(RM) -rf $(TARGET_DIR)/etc/openldap
|
|
endef
|
|
OPENLDAP_POST_INSTALL_TARGET_HOOKS += OPENLDAP_REMOVE_CLIENTS
|
|
endif
|
|
|
|
$(eval $(autotools-package))
|