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


Chris

-- 
cityweb DSL start ab monatlich 3,49 Euro *** http://www.cityweb.de/dsl ***
        Sekundenschneller Seitenaufbau, Downloads, Videos und vieles mehr. Mit
        Cityweb-DSL wird der Blick auf die Uhr beim Surfen �berfl�ssig!


-
List info/subscribe/unsubscribe? See http://www.freeradius.org/list/users.html

Reply via email to