Bug 55680 - [PATCH] Mk/bsd.port.mk: add OpenLDAP support
Summary: [PATCH] Mk/bsd.port.mk: add OpenLDAP support
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: Normal Affects Only Me
Assignee: Port Management Team
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-08-17 20:50 UTC by Oliver Eikemeier
Modified: 2003-08-27 05:44 UTC (History)
1 user (show)

See Also:


Attachments
bsd.port.mk.patch (1.46 KB, patch)
2003-08-17 20:50 UTC, Oliver Eikemeier
no flags Details | Diff
file.diff (1.04 KB, patch)
2003-08-17 20:50 UTC, Oliver Eikemeier
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Oliver Eikemeier 2003-08-17 20:50:16 UTC
This is a followup to PR 54436, a patch for bsd.port.mk that enables
ports that rely on the OpenLDAP client libraries to use USE_OPENLDAP=yes.
It was part of PR 54436 and is analogous to PR 55597.

More and more ports contain switching logic between the various OpenLDAP
ports, with different parameters. Moreover, if a port contains no
switching logic but depends on OpenLDAP 2.0 and another on OpenLDAP 2.1,
these ports become incompatible, because OpenLDAP 2.0 and OpenLDAP 2.1 are
mutually exclusive.

USE_OPENLDAP should remedy this situation.

If a port uses the OpenLDAP libraries a required a minumum OpenLDAP version,
the following usage is suggested:

USE_OPENLDAP=	yes
.if defined(WANT_OPENLDAP_VER) && ${WANT_OPENLDAP_VER} == 12
BROKEN=		"this port requires OpenLDAP >= 2.0"
.endif

An USE_OPENLDAP_VER switch is provided, but its usage is discouraged, and
it is not documented.

Fix: The logic should not be in net/openldap20-server/bsd.openldap.mk, it is
too misleading:

USE_OPENLDAP=yes
.include "${.CURDIR}/..../net/openldap20-server/bsd.openldap.mk"

results in a dependency on net/openldap21-client. Remove the file
and the references (thsi can be done before bsd.port.mk is modified):

rm net/openldap20-server/bsd.openldap.mk
Comment 1 Kirill Ponomarev freebsd_committer 2003-08-17 21:39:30 UTC
Responsible Changed
From-To: freebsd-ports-bugs->portmgr

portmgr territory
Comment 2 Joe Marcus Clarke freebsd_committer 2003-08-17 21:50:39 UTC
State Changed
From-To: open->analyzed

A patch that merges bsd.openldap.mk into bsd.port.mk is currently being tested 
on bento.
Comment 3 Joe Marcus Clarke freebsd_committer 2003-08-27 05:43:53 UTC
State Changed
From-To: analyzed->closed

Committed, thanks!