Le Thu, 22 Apr 2004 09:36:27 +0900 (JST)
Masao Uebayashi a ecrit :
> > Most notably, that means the following definitions were killed:
> > 
> > PINE_USE_LDAP          -> mail/pine
> > POSTFIX_USE_LDAP       -> mail/postfix{,-current}
> > SAMBA_USE_LDAP         -> net/{ja-samba,samba,samba2}
> > NNTP_USE_LDAP          -> news/nntpcache
> > 
> > and all replaced with USE_OPENLDAP.
> 
> How about testing USE_LDAP.${PKGBASE} and override USE_LDAP with
> USE_LDAP.${PKGBASE}?  This way users can override package specific
> values (e.g., "I don't want to use LDAP for Samba, but for others"...
> I don't know if such a case exists in real life, though), keeping
> variable name space cleaner.
> 
> One problem of this is that the overriding logic needs to be included
> prior to any reference of USE_LDAP.

While I understand the need, it is not very likely to happen often, and it
is always possible to do something like this in mk.conf:

.if !empty(.CURDIR:M*/net/*samba*)
USE_OPENLDAP=NO
.else
USE_OPENLDAP=YES
.endif

I think it should be considered, though, but first we need to make pkgsrc
more consistent.  I installed a mail server this month, using OpenLDAP as
the backend, and Postfix, dovecot, and also cyrus-saslauthd.  That meant
defining 3 variables to activate LDAP support in each...

-- 
Quentin Garnier - cube@cubidou.net - cube@NetBSD.org
"Feels like I'm fiddling while Rome is burning down.
Should I lay my fiddle down and take a rifle from the ground ?"
Leigh Nash/Sixpence None The Richer, Paralyzed, Divine Discontents, 2002.