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

Reply via email to