On Mon 10 Apr 2006 18:18, Joe Maimon wrote:
Peter Nixon wrote:
Eeeek.. Huge patch.. I get the feeling you have included more than necessary in this patch..
-Peter
On Mon 10 Apr 2006 18:03, Joe Maimon wrote:
--with-modules="rlm_sqlippool"
Well yes, it includes the changes to configure as well. Sorry about that. Strip it out and perform a rebuild locally before checking it in, if ever.
The actual patch to configure.in is cut'n'pasted here
diff -urNad freeradius-2.0.0~/configure.in freeradius-2.0.0/configure.in --- freeradius-2.0.0~/configure.in 2006-01-24 21:53:59.000000000 -0500 +++ freeradius-2.0.0/configure.in 2006-01-24 21:54:05.000000000 -0500 @@ -219,6 +219,14 @@ done ])
+MODULES= +AC_ARG_WITH(modules, +[ --with-modules=QUOTED-MODULE-LIST],[ + for i in $withval; do + MODULES="$MODULES $i" + done +]) + dnl # dnl # Enable developer C compiler warnings dnl # @@ -1018,7 +1026,6 @@ dnl ############################################################ dnl # make modules by list dnl ############################################################# -MODULES= if test "x$EXPERIMENTAL" = "xyes"; then for foo in `ls -1 src/modules | grep rlm_`; do MODULES="$MODULES $foo"
OK. I have committed it to cvs head. Is there any reason why it shouldn't go into the 1.1.x branch also? -- Peter Nixon http://www.peternixon.net/ PGP Key: http://www.peternixon.net/public.asc