On Mon, 14 Jul 2003 05:17 pm, Christian Esken wrote: > Alan DeKok wrote: > >Christian Esken <[EMAIL PROTECTED]> wrote: > >> PS: The reason why configure finds the lib, but not the include is > >> strange. It looks like gcc looks at /usr/local/gnu/lib by default , > >> but the preprocessor (gpp) does not look at /usr/local/gnu/include > > > > So your compiler & associated tools can't find some gdbm > >information, and you're blaming FreeRADIUS, why, exactly? > > I am fully aware I can set appropriate environment variables before > compiling (to set extra inlcude/lib paths). This is not the point here. > > The point is that gdbm.h is not there and the configure.in of the > rlm_counter module does not care. So the FreeRADIUS build system tries to > compile the module, which fails due to the missing gdbm.h include. I think > this is bad behaviour. Nice behaviour would be to say: "Not compiling > rlm_counter (gdbm.h missing)". > > So the configure.in should be fixed. I am currently testing the follwing > fix, but expecting it to fail (I am in no ways an autoconf expert.). > > [EMAIL PROTECTED] ->diff -u configure.in~ configure.in > --- configure.in~ Mon Feb 25 19:44:37 2002 > +++ configure.in Mon Jul 14 15:32:58 2003 > @@ -8,6 +8,9 @@ > AC_PROG_CPP > > AC_SMART_CHECK_INCLUDE(gdbm.h) > + if test "x$ac_cv_header_gdbm_h" != "xyes"; then > + fail="$fail gdbmh.h" > + fi > AC_SMART_CHECK_LIB(gdbm, gdbm_open) > if test "x$ac_cv_lib_gdbm_gdbm_open" != "xyes"; then > fail="$fail libgdbm" > > > I am fully aware that it is only a flaw of the rlm_counter module. But > including a module for compiling with knowing compiling WILL fail is not > good. So I thought you might be interested
A similar problem exists in the kerberos module. I expect we will have these issues sorted out for version 1.0 (or ever version 0.9.1) Send me a patch that works and I will test it :-) -- Peter Nixon http://www.peternixon.net/ PGP Key: http://www.peternixon.net/public.asc - List info/subscribe/unsubscribe? See http://www.freeradius.org/list/users.html
