freeradius2 port with e-dir

[Olli Hauer]

On 2010-09-17 22:00, John D McDonnell wrote:
> Around 3 weeks ago, I had asked on the list about how to compile 
> FreeRADIUS 2 with support for e-directory. I had been hoping to get an 
> answer back by now from someone with a success story of using FreeRADIUS
> 2 
> on FreeBSD with Novell E-Directory for a backend. With the (very) old 
> freeradius1 port, there is a toggle in options for e-dir support, but
> that 
> is missing in the freeradius2 port.
> 
> It was verified by the FreeRADIUS mailing list that support for 
> E-Directory needs to be compiled in. I've not had a chance to set up the 
> new server yet, but that should be happening in the next week or two.
> With 
> the imminent release of 2.1.10, I may just set the rest of the server up 
> and wait for that to install FreeRADIUS.
> 
> I tried looking through the Makefile but I didn't find any reference to 
> edir there. Would it be possible to add the E-Directory option back or
> at 
> least provide the option within the Makefile even if not available from 
> make config? (Or just tell me I'm an idiot and show me what I missed
> that 
> gets it to build with e-dir support. ;))
> 
> Thanks in advance.
> 

Hi John,

can you test the attached patch for freeradius2.

The --with-edir parameter is only used in the
"src/modules/rlm_ldap/configure" script, I guess it was just
overlooked.

If the patch is working for you please open a PR.

--
olli
-------------- next part --------------
Index: Makefile
===================================================================
RCS file: /home/pcvs/ports/net/freeradius2/Makefile,v
retrieving revision 1.88
diff -u -r1.88 Makefile
--- Makefile	15 Sep 2010 18:34:58 -0000	1.88
+++ Makefile	17 Sep 2010 22:56:06 -0000
@@ -47,6 +47,7 @@
 		KERBEROS	"With Kerberos support" off \
 		HEIMDAL		"With Heimdal Kerberos support" off \
 		LDAP		"With LDAP database support" off \
+		EDIR		"With Novell eDirectory support" off \
 		MYSQL		"With MySQL database support" off \
 		PGSQL		"With PostgreSQL database support" off \
 		UNIXODBC	"With unixODBC database support" off \
@@ -97,6 +98,10 @@
 PLIST_SUB+=	KRB5="@comment "
 .endif
 
+.if defined(WITH_EDIR) && !defined(WITH_LDAP)
+WITH_LDAP=	yes
+.endif
+
 .ifdef(WITH_LDAP)
 USE_OPENLDAP=	YES
 CONFIGURE_ARGS+=--with-rlm_ldap
@@ -107,6 +112,10 @@
 PLIST_SUB+=	LDAP="@comment "
 .endif
 
+.ifdef(WITH_EDIR)
+CONFIGURE_ARGS+=--with-edir
+.endif
+
 .ifdef(WITH_MYSQL)
 USE_MYSQL=	YES
 CONFIGURE_ARGS+=--with-rlm_sql_mysql