Ilguiz Latypov <[EMAIL PROTECTED]> wrote:
> The bootstrap script allows to remove automatically generated files from
> the CVS repository. However, some constructs in the updated configure.in
> are valid only with the newer autoconf. Perhaps, putting AC_PREREQ(2.53)
> into configure.in could be a good precaution.
Maybe. I'd rather not force the use of the "latest and greatest"
tools, as that usually creates more problems than it solves.
> Another small problem was due to the sql_fetch_row() prototypes in
...
> [ rlm_sql_postgresql and rlm_sql_unixodbc ]
Hmm... I don't even understand why those functions aren't "static".
They should ALL be "static", and they shouldn't have prototypes in the
header file.
> not synchronized with their .c counterparts. Does the patch included
> below fix those prototypes correctly? Thank you,
Yes, I've applied those patches, thanks.
> +AM_INIT_AUTOMAKE(radiusd, ${RADIUSD_VERSION})
I *hate* automake. It's a useless, insane, bloated piece of
garbage. I've *never* found a case where using automake helped. It's
always made the source harder to maintain, and reduced previously neat
Makefiles to opaque strings typed by dyslexic monkeys on acid.
Alan DeKok.
-
List info/subscribe/unsubscribe? See http://www.freeradius.org/list/users.html