Skip to content

Commit

Permalink
net/openldap24-server: Make SASL permanent for OpenLDAP port.
Browse files Browse the repository at this point in the history
PR:		ports/257374
Reviewed by:	obrien
Approved by:	portmgr (exp-run by antoine)
Differential Revision: https://reviews.freebsd.org/D31301
  • Loading branch information
delphij committed Aug 1, 2021
1 parent f120f8c commit 974e13b
Show file tree
Hide file tree
Showing 34 changed files with 65 additions and 190 deletions.
5 changes: 5 additions & 0 deletions CHANGES
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,11 @@ in the release notes and/or placed into UPDATING.

All ports committers are allowed to commit to this file.

20210801:
AUTHOR: delphij@FreeBSD.org
WANT_OPENLDAP_SASL is removed now that OpenLDAP is always built with
SASL support.

20210730:
AUTHOR: tcberner@FreeBSD.org

Expand Down
1 change: 1 addition & 0 deletions MOVED
Original file line number Diff line number Diff line change
Expand Up @@ -16474,3 +16474,4 @@ databases/rubygem-redis-namespace17|databases/rubygem-redis-namespace|2021-07-25
multimedia/mlt|multimedia/mlt6|2021-07-30|Moved to a versioned directory to make place for future versions
multimedia/mlt-qt5|multimedia/mlt6-qt5|2021-07-30|Moved to a versioned directory to make place for future versions
multimedia/py-mlt|multimedia/py-mlt6|2021-07-30|Moved to a versioned directory to make place for future versions
net/openldap24-sasl-client|net/openldap24-client|2021-08-01|SASL is always enabled for OpenLDAP
13 changes: 1 addition & 12 deletions Mk/bsd.ldap.mk
Original file line number Diff line number Diff line change
Expand Up @@ -20,9 +20,6 @@ Database_Include_MAINTAINER= ports@FreeBSD.org
# Default: 24.
# WANT_OPENLDAP_VER
# - Maintainer can set an arbitrary version of OpenLDAP by using it.
# WANT_OPENLDAP_SASL
# - User-defined variable to depend upon SASL-enabled OpenLDAP
# client. Must NOT be set in a port Makefile.
# IGNORE_OPENLDAP_OPENLDAP
# - This variable can be defined if the ports doesn't support
# one or more version of OpenLDAP.
Expand Down Expand Up @@ -64,14 +61,6 @@ IGNORE= cannot install: OpenLDAP versions mismatch: openldap${_OPENLDAP_VER}-cli
CFLAGS+= -DLDAP_DEPRECATED

_OPENLDAP_CLIENT_PKG!= ${PKG_INFO} -Ex openldap.\*-client 2>/dev/null; ${ECHO_CMD}
_OPENLDAP_FLAVOUR= ${_OPENLDAP_CLIENT_PKG:C/openldap//:C/-client-.*//}

.if defined(WANT_OPENLDAP_SASL)
.if !empty(_OPENLDAP_CLIENT_PKG) && empty(_OPENLDAP_FLAVOUR)
IGNORE= cannot install: SASL support requested and ${_OPENLDAP_CLIENT_PKG} is installed
.endif
_OPENLDAP_FLAVOUR= -sasl
.endif

# And now we are checking if we can use it
.if defined(OPENLDAP${OPENLDAP_VER}_LIB)
Expand All @@ -86,7 +75,7 @@ IGNORE= cannot install: doesn't work with OpenLDAP version: ${OPENLDAP_VER} (Do
. endif
. endfor
.endif # IGNORE_WITH_OPENLDAP
LIB_DEPENDS+= ${OPENLDAP${OPENLDAP_VER}_LIB}:net/openldap${OPENLDAP_VER}${_OPENLDAP_FLAVOUR}-client
LIB_DEPENDS+= ${OPENLDAP${OPENLDAP_VER}_LIB}:net/openldap${OPENLDAP_VER}-client
.else
IGNORE= cannot install: unknown OpenLDAP version: ${OPENLDAP_VER}
.endif # Check for correct libs
Expand Down
5 changes: 1 addition & 4 deletions Mk/bsd.port.mk
Original file line number Diff line number Diff line change
Expand Up @@ -364,11 +364,8 @@ FreeBSD_MAINTAINER= portmgr@FreeBSD.org
# USE_OPENLDAP - If set, this port uses the OpenLDAP libraries.
# Implies: WANT_OPENLDAP_VER?=24
# WANT_OPENLDAP_VER
# - Legal values are: 23, 24
# - Legal values are: 24
# If set to an unknown value, the port is marked BROKEN.
# WANT_OPENLDAP_SASL
# - If set, the system should use OpenLDAP libraries
# with SASL support.
##
# USE_JAVA - If set, this port relies on the Java language.
# Implies inclusion of bsd.java.mk. (Also see
Expand Down
13 changes: 13 additions & 0 deletions UPDATING
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,19 @@ they are unavoidable.
You should get into the habit of checking this file for changes each time
you update your ports collection, before attempting any port upgrades.

20210801:
AFFECTS: users of OpenLDAP
AUTHOR: delphij@FreeBSD.org

SASL is now always enabled for OpenLDAP.

If you use portmaster:
portmaster -o net/openldap24-client openldap24-sasl-client
If you use portupgrade:
portupgrade -fo net/openldap24-client openldap24-sasl-client
If you use pkg with binary packages:
pkg set -o net/openldap24-sasl-client:net/openldap24-client

20210730:
AFFECTS: users of multimedia/mlt, multimedia/mlt-qt and multimedia/py-mlt
AUTHOR: kde@FreeBSD.org
Expand Down
4 changes: 2 additions & 2 deletions databases/mysql57-client/Makefile
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
# Created by: "Mahdi Mokhtari <mokhi64@gmail.com>"

PORTNAME= mysql
PORTREVISION= 0
PORTREVISION= 1
PKGNAMESUFFIX= 57-client

COMMENT= Multithreaded SQL database (client)
Expand Down Expand Up @@ -36,7 +36,7 @@ OPTIONS_GROUP_PLUGINS= SASLCLIENT
SASLCLIENT_DESC= SASL client plugin module
SASLCLIENT_CMAKE_BOOL= WITH_AUTHENTICATION_LDAP
SASLCLIENT_LIB_DEPENDS= libsasl2.so:security/cyrus-sasl2
SASLCLIENT_BUILD_DEPENDS= ${LOCALBASE}/include/lber.h:net/openldap24-sasl-client
SASLCLIENT_USE= OPENLDAP=yes
OPTIONS_DEFAULT+= SASLCLIENT

# issue 166367: adding symlinks for back-compatibility with ${lib}_r
Expand Down
11 changes: 0 additions & 11 deletions databases/mysql57-server/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -164,17 +164,6 @@ CMAKE_ARGS+= -DWITH_SSL=${OPENSSLBASE}
CXXFLAGS+= -fpermissive
.endif

# mysql57-server fails to compile with llvm11, which was imported to current in r364284
# without a version increase (1300109 = r364274).
# Until a proper fix is provided, simply use llvm from ports, which probably could be
# a backport of amongst other tings:
# https://github.com/mysql/mysql-server/commit/08f46b3c00ee70e7ed7825daeb91df2289f80f50
.if ${CHOSEN_COMPILER_TYPE} == clang && ${OPSYS} == FreeBSD && ${OSVERSION} >= 1300109
BUILD_DEPENDS+= clang${LLVM_DEFAULT}:devel/llvm${LLVM_DEFAULT}
CC= ${LOCALBASE}/bin/clang${LLVM_DEFAULT}
CXX= ${LOCALBASE}/bin/clang++${LLVM_DEFAULT}
.endif

.if ${ARCH} == powerpc
LLD_UNSAFE= yes
.endif
Expand Down
4 changes: 2 additions & 2 deletions databases/percona57-client/Makefile
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
# Created by: Mark Felder <feld@FreeBSD.org>

PORTNAME= percona
PORTREVISION?= 0
PORTREVISION?= 1
PKGNAMESUFFIX= 57-client

COMMENT= Multithreaded SQL database (client)
Expand Down Expand Up @@ -36,7 +36,7 @@ OPTIONS_GROUP_PLUGINS= SASLCLIENT
SASLCLIENT_DESC= SASL client plugin module
SASLCLIENT_CMAKE_BOOL= WITH_AUTHENTICATION_LDAP
SASLCLIENT_LIB_DEPENDS= libsasl2.so:security/cyrus-sasl2
SASLCLIENT_BUILD_DEPENDS= ${LOCALBASE}/include/lber.h:net/openldap24-sasl-client
SASLCLIENT_USE= OPENLDAP=yes
OPTIONS_DEFAULT+= SASLCLIENT
OPTIONS_SUB= yes

Expand Down
2 changes: 1 addition & 1 deletion lang/php73/Makefile
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
PORTNAME= php73
DISTVERSION= 7.3.29
PORTREVISION?= 0
PORTREVISION?= 1
CATEGORIES?= lang devel www
MASTER_SITES= PHP/distributions
DISTNAME= php-${DISTVERSION}
Expand Down
6 changes: 1 addition & 5 deletions lang/php73/Makefile.ext
Original file line number Diff line number Diff line change
Expand Up @@ -174,13 +174,9 @@ CONFIGURE_ARGS+=--enable-json
.endif

.if ${PHP_MODNAME} == "ldap"
CONFIGURE_ARGS+=--with-ldap=${LOCALBASE}
CONFIGURE_ARGS+=--with-ldap=${LOCALBASE} --with-ldap-sasl=${LOCALBASE}

USE_OPENLDAP= yes

. ifdef(WANT_OPENLDAP_SASL)
CONFIGURE_ARGS+=--with-ldap-sasl=${LOCALBASE}
. endif
.endif

.if ${PHP_MODNAME} == "mbstring"
Expand Down
2 changes: 1 addition & 1 deletion lang/php74/Makefile
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
PORTNAME= php74
DISTVERSION= 7.4.22
PORTREVISION?= 0
PORTREVISION?= 1
CATEGORIES?= lang devel www
MASTER_SITES= PHP/distributions
DISTNAME= php-${DISTVERSION}
Expand Down
6 changes: 1 addition & 5 deletions lang/php74/Makefile.ext
Original file line number Diff line number Diff line change
Expand Up @@ -172,13 +172,9 @@ BUILD_DEPENDS= re2c:devel/re2c
.endif

.if ${PHP_MODNAME} == "ldap"
CONFIGURE_ARGS+=--with-ldap=${LOCALBASE}
CONFIGURE_ARGS+=--with-ldap=${LOCALBASE} --with-ldap-sasl=${LOCALBASE}

USE_OPENLDAP= yes

. ifdef(WANT_OPENLDAP_SASL)
CONFIGURE_ARGS+=--with-ldap-sasl=${LOCALBASE}
. endif
.endif

.if ${PHP_MODNAME} == "mbstring"
Expand Down
2 changes: 1 addition & 1 deletion lang/php80/Makefile
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
PORTNAME= php80
DISTVERSION= 8.0.9
PORTREVISION?= 0
PORTREVISION?= 1
CATEGORIES?= lang devel www
MASTER_SITES= PHP/distributions
DISTNAME= php-${DISTVERSION}
Expand Down
6 changes: 1 addition & 5 deletions lang/php80/Makefile.ext
Original file line number Diff line number Diff line change
Expand Up @@ -172,13 +172,9 @@ BUILD_DEPENDS= re2c:devel/re2c
.endif

.if ${PHP_MODNAME} == "ldap"
CONFIGURE_ARGS+=--with-ldap=${LOCALBASE}
CONFIGURE_ARGS+=--with-ldap=${LOCALBASE} --with-ldap-sasl=${LOCALBASE}

USE_OPENLDAP= yes

. ifdef(WANT_OPENLDAP_SASL)
CONFIGURE_ARGS+=--with-ldap-sasl=${LOCALBASE}
. endif
.endif

.if ${PHP_MODNAME} == "mbstring"
Expand Down
6 changes: 1 addition & 5 deletions mail/opendkim/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@

PORTNAME= opendkim
PORTVERSION= 2.10.3
PORTREVISION= 11
PORTREVISION= 12
CATEGORIES= mail security
MASTER_SITES= SF/${PORTNAME} \
SF/${PORTNAME}/Previous%20Releases \
Expand Down Expand Up @@ -107,10 +107,6 @@ MAKE_ENV= INSTALL_STRIP_FLAG=${STRIP}
WITH_DEBUG= yes
.endif

.if ${PORT_OPTIONS:MSASL} && ${PORT_OPTIONS:MOPENLDAP}
WANT_OPENLDAP_SASL= yes
.endif

.if ${PORT_OPTIONS:MFILTER}
SUB_FILES= pkg-message
WITHOUT_MILTER_CFLAGS= yes
Expand Down
13 changes: 3 additions & 10 deletions mail/postfix-current/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@

PORTNAME= postfix
DISTVERSION= 3.7-20210717
PORTREVISION?= 0
PORTREVISION?= 1
PORTEPOCH= 6
CATEGORIES= mail
MASTER_SITES= http://ftp.porcupine.org/mirrors/postfix-release/ \
Expand Down Expand Up @@ -38,7 +38,7 @@ USE_SUBMAKE= yes
SCRIPTS_ENV+= POSTFIX_DEFAULT_MTA="${POSTFIX_DEFAULT_MTA}"

OPTIONS_SUB= yes
OPTIONS_DEFINE= BDB BLACKLISTD CDB DOCS EAI INST_BASE LDAP LDAP_SASL \
OPTIONS_DEFINE= BDB BLACKLISTD CDB DOCS EAI INST_BASE LDAP \
LMDB MYSQL NIS PCRE PGSQL SASL SQLITE TEST TLS

OPTIONS_RADIO= RG1
Expand All @@ -63,7 +63,6 @@ CDB_DESC= CDB maps lookups
EAI_DESC= Email Address Internationalization (SMTPUTF8) support
INST_BASE_DESC= Install into /usr and /etc/postfix
LDAP_DESC= LDAP maps (uses WITH_OPENLDAP_VER)
LDAP_SASL_DESC= LDAP client-to-server SASL auth
LMDB_DESC= LMDB maps
TEST_DESC= SMTP/LMTP test server and generator

Expand Down Expand Up @@ -234,15 +233,9 @@ DYN_EXT+= ldap
WANT_OPENLDAP_VER= ${WITH_OPENLDAP_VER}
. endif
POSTFIX_CCARGS+= -DHAS_LDAP -I${LOCALBASE}/include
POSTFIX_CCARGS+= -I${LOCALBASE}/include/sasl -DUSE_LDAP_SASL
POSTFIX_DYN_AUXLIBS+= "AUXLIBS_LDAP=-L${LOCALBASE}/lib -lldap -L${LOCALBASE}/lib -llber"
_REQUIRE+= slapd
. if ${PORT_OPTIONS:MLDAP_SASL}
. if ! ${PORT_OPTIONS:MSASL}
LIB_DEPENDS+= libsasl2.so:security/cyrus-sasl2
. endif
WANT_OPENLDAP_SASL= yes
POSTFIX_CCARGS+= -I${LOCALBASE}/include/sasl -DUSE_LDAP_SASL
. endif
.endif

.if ${PORT_OPTIONS:MCDB}
Expand Down
13 changes: 3 additions & 10 deletions mail/postfix/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@

PORTNAME= postfix
DISTVERSION= 3.6.2
PORTREVISION?= 0
PORTREVISION?= 1
PORTEPOCH= 1
CATEGORIES= mail
MASTER_SITES= http://ftp.porcupine.org/mirrors/postfix-release/ \
Expand Down Expand Up @@ -38,7 +38,7 @@ USE_SUBMAKE= yes
SCRIPTS_ENV+= POSTFIX_DEFAULT_MTA="${POSTFIX_DEFAULT_MTA}"

OPTIONS_SUB= yes
OPTIONS_DEFINE= BDB BLACKLISTD CDB DOCS EAI INST_BASE LDAP LDAP_SASL \
OPTIONS_DEFINE= BDB BLACKLISTD CDB DOCS EAI INST_BASE LDAP \
LMDB MYSQL NIS PCRE PGSQL SASL SQLITE TEST TLS

OPTIONS_RADIO= RG1
Expand All @@ -63,7 +63,6 @@ CDB_DESC= CDB maps lookups
EAI_DESC= Email Address Internationalization (SMTPUTF8) support
INST_BASE_DESC= Install into /usr and /etc/postfix
LDAP_DESC= LDAP maps (uses WITH_OPENLDAP_VER)
LDAP_SASL_DESC= LDAP client-to-server SASL auth
LMDB_DESC= LMDB maps
TEST_DESC= SMTP/LMTP test server and generator

Expand Down Expand Up @@ -234,15 +233,9 @@ DYN_EXT+= ldap
WANT_OPENLDAP_VER= ${WITH_OPENLDAP_VER}
. endif
POSTFIX_CCARGS+= -DHAS_LDAP -I${LOCALBASE}/include
POSTFIX_CCARGS+= -I${LOCALBASE}/include/sasl -DUSE_LDAP_SASL
POSTFIX_DYN_AUXLIBS+= "AUXLIBS_LDAP=-L${LOCALBASE}/lib -lldap -L${LOCALBASE}/lib -llber"
_REQUIRE+= slapd
. if ${PORT_OPTIONS:MLDAP_SASL}
. if ! ${PORT_OPTIONS:MSASL}
LIB_DEPENDS+= libsasl2.so:security/cyrus-sasl2
. endif
WANT_OPENLDAP_SASL= yes
POSTFIX_CCARGS+= -I${LOCALBASE}/include/sasl -DUSE_LDAP_SASL
. endif
.endif

.if ${PORT_OPTIONS:MCDB}
Expand Down
13 changes: 3 additions & 10 deletions mail/postfix35/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@

PORTNAME= postfix
DISTVERSION= 3.5.12
PORTREVISION?= 0
PORTREVISION?= 1
PORTEPOCH= 1
CATEGORIES= mail
MASTER_SITES= http://ftp.porcupine.org/mirrors/postfix-release/ \
Expand Down Expand Up @@ -41,7 +41,7 @@ USE_SUBMAKE= yes
SCRIPTS_ENV+= POSTFIX_DEFAULT_MTA="${POSTFIX_DEFAULT_MTA}"

OPTIONS_SUB= yes
OPTIONS_DEFINE= BDB BLACKLISTD CDB DOCS EAI INST_BASE LDAP LDAP_SASL \
OPTIONS_DEFINE= BDB BLACKLISTD CDB DOCS EAI INST_BASE LDAP \
LMDB MYSQL NIS PCRE PGSQL SASL SQLITE TEST TLS

OPTIONS_RADIO= RG1
Expand All @@ -67,7 +67,6 @@ CDB_DESC= CDB maps lookups
EAI_DESC= Email Address Internationalization (SMTPUTF8) support
INST_BASE_DESC= Install into /usr and /etc/postfix
LDAP_DESC= LDAP maps (uses WITH_OPENLDAP_VER)
LDAP_SASL_DESC= LDAP client-to-server SASL auth
LMDB_DESC= LMDB maps
TEST_DESC= SMTP/LMTP test server and generator

Expand Down Expand Up @@ -230,15 +229,9 @@ DYN_EXT+= ldap
WANT_OPENLDAP_VER= ${WITH_OPENLDAP_VER}
. endif
POSTFIX_CCARGS+= -DHAS_LDAP -I${LOCALBASE}/include
POSTFIX_CCARGS+= -I${LOCALBASE}/include/sasl -DUSE_LDAP_SASL
POSTFIX_DYN_AUXLIBS+= "AUXLIBS_LDAP=-L${LOCALBASE}/lib -lldap -L${LOCALBASE}/lib -llber"
_REQUIRE+= slapd
. if ${PORT_OPTIONS:MLDAP_SASL}
. if ! ${PORT_OPTIONS:MSASL}
LIB_DEPENDS+= libsasl2.so:security/cyrus-sasl2
. endif
WANT_OPENLDAP_SASL= yes
POSTFIX_CCARGS+= -I${LOCALBASE}/include/sasl -DUSE_LDAP_SASL
. endif
.endif

.if ${PORT_OPTIONS:MCDB}
Expand Down
8 changes: 2 additions & 6 deletions mail/vpopmail/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@

PORTNAME= vpopmail
PORTVERSION= 5.4.33
PORTREVISION= 3
PORTREVISION= 4
CATEGORIES= mail
MASTER_SITES= SF/${PORTNAME}/${PORTNAME}-stable/${PORTVERSION}

Expand Down Expand Up @@ -68,7 +68,7 @@ OPTIONS_DEFINE= DOCS \
SQL_LOG \
SQL_LOG_TRIM
OPTIONS_SINGLE= AUTH
OPTIONS_SINGLE_AUTH= CDB MYSQL PGSQL LDAP LDAP_SASL ORACLE SYBASE
OPTIONS_SINGLE_AUTH= CDB MYSQL PGSQL LDAP ORACLE SYBASE

OPTIONS_DEFAULT=MD5_PASSWORDS \
CDB \
Expand All @@ -90,7 +90,6 @@ MYSQL_REPLICATION_DESC= MySQL database replication support
MYSQL_LIMITS_DESC= MySQL mailbox limitations support
ORACLE_DESC= Auth via Oracle
LDAP_DESC= Auth via LDAP
LDAP_SASL_DESC= Auth via LDAP SASL
VALIAS_DESC= Store aliases in DB instead of .qmail files
ROAMING_DESC= roaming users support
IP_ALIAS_DESC= IP alias support
Expand Down Expand Up @@ -255,9 +254,6 @@ VPOPMAIL_DIR?= ${PREFIX}/vpopmail

.if ${PORT_OPTIONS:MLDAP}
USE_OPENLDAP= yes
.if ${PORT_OPTIONS:MLDAP_SASL}
WANT_OPENLDAP_SASL= yes
.endif
LDAP_FILES= ${WRKSRC}/doc/README.ldap \
${WRKSRC}/ldap/nsswitch.conf \
${WRKSRC}/ldap/pam_ldap.conf \
Expand Down

0 comments on commit 974e13b

Please sign in to comment.