Re: pkg/39386: www/squid does not support LDAP authentication

[Edgar Fuß <ef%math.uni-bonn.de@localhost>]

The following reply was made to PR pkg/39386; it has been noted by GNATS.

From: =?ISO-8859-1?Q?Edgar_Fu=DF?= <ef%math.uni-bonn.de@localhost>
To: gnats-bugs%NetBSD.org@localhost
Cc: 
Subject: Re: pkg/39386: www/squid does not support LDAP authentication
Date: Wed, 20 Aug 2008 15:10:32 +0200

 Thanks for looking into this.
 
 > Your patch introduce[s] dependency to database/openldap-client to  
 > squid package unconditionally.
 Oops? I thought that
 
 .if !empty(SQUID_BASIC_AUTH_HELPERS:MLDAP)
 .include "../../databases/openldap-client/buildlink3.mk"
 .endif
 
 would just introduce the dependency if SQUID_BASIC_AUTH_HELPERS  
 contained LDAP?
 
 > I think it should be option [...]
 Something like
 
 .if !empty(PKG_OPTIONS:Mldap)
 .include "../../databases/openldap-client/buildlink3.mk"
 SQUID_BASIC_AUTH_HELPERS+= LDAP
 .endif