ports/119986: print/cups-base needs LDAP option to expose hidden dependency

[Virgil Champlin]

>Number:         119986
>Category:       ports
>Synopsis:       print/cups-base needs LDAP option to expose hidden dependency
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    freebsd-ports-bugs
>State:          open
>Quarter:        
>Keywords:       
>Date-Required:
>Class:          change-request
>Submitter-Id:   current-users
>Arrival-Date:   Fri Jan 25 23:10:04 UTC 2008
>Closed-Date:
>Last-Modified:
>Originator:     Virgil Champlin
>Release:        FreeBSD 6.2-RELEASE-p10 i386
>Organization:
Stupidog.org
>Environment:
System: FreeBSD whisper.stupidog.org 6.2-RELEASE-p10 FreeBSD 6.2-RELEASE-p10 #12: Tue Jan 15 09:38:13 PST 2008 root at whisper.stupidog.org:/usr/obj/usr/src/sys/STUPIDOG i386

>Description:
	The CUPS source configures with --enable-ldap as a default and will
	build with LDAP support if it finds ldap.h and an ldap library.  If
	you build the cups-base package on a machine that has LDAP support,
	the resulting package will have a hidden dependency on the ldap
	library which prevents it from running on a system without LDAP
	support.  e.g. It will successfully install on a host that has no
	LDAP support but yields the following on start-up:
	    /libexec/ld-elf.so.1: Shared object "libldap-2.3.so.2" not
	    found, required by "cupsd"
	This is common in environments with a single build machine that
	prepares packages for other local hosts.
>How-To-Repeat:
	Build a print/cups-base package on a host with openldap-client.
	Install the resulting package on a host that has no LDAP support.
	'sudo /usr/local/etc/rc.d/cupsd start' will return the above
	shared library error.
>Fix:
	The following patch to print/cups-base/Makefile (cvs 1.85) will
	expose the ldap option at the port level and allow the ldap
	dependency, if enabled, to be recognized.  Thanks much.  -virgil

--- Makefile.patch begins here ---
--- Makefile.orig	Wed Jan 23 08:00:56 2008
+++ Makefile	Wed Jan 23 13:28:14 2008
@@ -49,7 +49,8 @@
 		PYTHON		"Build PYTHON support" off \
 		LIBPAPER	"Build with libpaper support" off \
 		DNSSD		"Build with DNS_SD (avahi) support" off \
-		PAM		"Build with PAM support" off
+		PAM		"Build with PAM support" off \
+		LDAP		"Build with ldap support" off
 
 CUPS_ETCDIR=	${PREFIX}/etc/cups
 CUPS_SPOOLDIR=	/var/spool/cups
@@ -111,6 +112,13 @@
 .else
 CONFIGURE_ARGS+=	--disable-pam
 PLIST_SUB+=		WITH_PAMD="@comment "
+.endif
+
+.if defined(WITH_LDAP)
+USE_OPENLDAP=		yes	
+CONFIGURE_ARGS+=	--enable-ldap
+.else
+CONFIGURE_ARGS+=	--disable-ldap
 .endif
 
 MAN1EXT=	1
--- Makefile.patch ends here ---


>Release-Note:
>Audit-Trail:
>Unformatted: