Re: ldap patch for samba port

[Juan Vera <juan_(_at_)_coresecurity_(_dot_)_com>]

Lukas Meyer wrote:

> Lukas Meyer wrote:
>
> > Tero Ripattila wrote:
> >
> > > -----BEGIN PGP SIGNED MESSAGE-----
> > >
> > > Hello Lukas,
> > >
> > > Wednesday, February 25, 2004, 10:52:55 AM, you [LK] wrote:
> > >
> > > LM> I'm looking for the maintainer of net/samba. I needed ldap 
> > > support in
> > > LM> samba so I added a flavor ldap to the Makefile.
> > >
> > > Good point, I needed to do the very same thing day before yesterday. I
> > > also needed to get a cups-enabled version of samba, because cups 
> > > support
> > > is disabled by default.
> > >
> > > I think something like as follows should be added to the Makefile too:
> > >
> > > FLAVORS=cups
> > >
> > > .if ${FLAVOR:L:Mcups}
> > > CONFIGURE_ARGS+= --enable-cups
> > > .else
> > > CONFIGURE_ARGS+= --disable-cups
> > > .endif
> > >
> > > Best regards, Tero
> > >
> > > -----BEGIN PGP SIGNATURE-----
> > > Version: PGP SDK 3.0.3
> > > Comment: ""
> > >
> > > iQCVAwUBQD0I96FEeQ41vAczAQFdnwP+JASUmswqdvLTVPc9MwYZ14DUWjtQY6G4
> > > SCo4ewffOLERzbVf2u+ZJs8dvzXlrpnpVSTGcWt532UrbxBmkPeDmkue0/BSJarY
> > > 8OZFWnBAAPK7voYC000imYViTHLihBpD31QlIsuEVcx40OToZlKzCaik+3gEbcGM
> > > Y5nbZcBVNpA=
> > > =AmQ+
> > > -----END PGP SIGNATURE-----
> > >  
> > I've added the cups flavor to net/samba too. This new net/samba port 
> > will reside in my ports tree until the maintainer could add this to 
> > his port. Is there actually anyone who maintains this port?
> >
> > regards
> > Lukas
> 34,37d33
> < FLAVORS=        ldap cups
> < FLAVOR?=
> <
> <
> 55,64d50
> <
> < .if ${FLAVOR:L:Mldap}
> < CONFIGURE_ARGS+= --with-ldapsam
> < .endif
> <
> < .if ${FLAVOR:L:Mcups}
> < CONFIGURE_ARGS+= --enable-cups
> < .else
> < CONFIGURE_ARGS+= --disable-cups
> < .endif

ldap FLAVOR is missing deps, it should read:

.if ${FLAVOR:L:Mldapsam}
LIB_DEPENDS+=   ldap.2,lber:openldap-client-2.*:databases/openldap
CONFIGURE_ARGS+=       --with-ldapsam
.endif