Re: 0.8.1 , src/modules/rlm_sql/drivers/rlm_sql_postgresql/sql_postgresql.c

2003-07-16 Thread Peter Nixon
Eeeek. I have no idea what you have done here :-)

try using the latest Prerelease version from:
 ftp://ftp.freeradius.org/pub/radius/freeradius-0.9.0-pre3.tar.gz

Let us know how you go..

Peter

On Wed, 16 Jul 2003 04:53 pm, Ali Gunduz wrote:
> Sorry if this is reported before...
>
> I'm having these errors while compiling 0.8.1 with postgresql support.
>
> #
> sql_postgresql.c:115:31: missing terminating " character
> sql_postgresql.c: In function `sql_check_error':
> sql_postgresql.c:116: error: parse error before "s"
> sql_postgresql.c:116:22: missing terminating " character
> sql_postgresql.c:129:31: missing terminating " character
> sql_postgresql.c:130: error: parse error before "s"
> sql_postgresql.c:130:11: missing terminating " character
> sql_postgresql.c: In function `sql_query':
> sql_postgresql.c:227: warning: implicit declaration of function
> `sql_store_result'
> sql_postgresql.c:228: warning: implicit declaration of function
> `sql_num_fields'
> sql_postgresql.c: At top level:
> sql_postgresql.c:257: warning: `sql_store_result' was declared
> implicitly `extern' and later `static'
> sql_postgresql.c:227: warning: previous declaration of
> `sql_store_result'
> sql_postgresql.c:274: warning: `sql_num_fields' was declared implicitly
> `extern' and later `static'
> sql_postgresql.c:228: warning: previous declaration of `sql_num_fields'
> make[10]: *** [sql_postgresql.o] Error 1
> #---
>
>
> Mentioned file is:
> src/modules/rlm_sql/drivers/rlm_sql_postgresql/sql_postgresql.c
>
> Line 115 of this file is:
> radlog(L_DBG, "rlm_sql_postgresql: Postgresql
> check_error:
> s, returning SQL_DOWN", PQresStatus(error));
>
> (there's a newline at the end of first line.. Above code segment is
> actually two lines in sql_postgresql.c)
>
> Guess it'd be:
> radlog(L_DBG, "rlm_sql_postgresql: Postgresql
> check_error: %s, returning SQL_DOWN", PQresStatus(error));
>
>
>
> The same mistake exists on line 128 too...
>
>
> -
> List info/subscribe/unsubscribe? See
> http://www.freeradius.org/list/users.html

-- 

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


Re: 0.8.1 , src/modules/rlm_sql/drivers/rlm_sql_postgresql/sql_postgresql.c

2003-07-16 Thread Alan DeKok
"Ali Gunduz" <[EMAIL PROTECTED]> wrote:
> I'm having these errors while compiling 0.8.1 with postgresql support.

  It's a bug in 0.8.1, which is fixed in 0.9.0.

  Alan DeKok.

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


0.8.1 , src/modules/rlm_sql/drivers/rlm_sql_postgresql/sql_postgresql.c

2003-07-16 Thread Ali Gunduz
Sorry if this is reported before...

I'm having these errors while compiling 0.8.1 with postgresql support.

#
sql_postgresql.c:115:31: missing terminating " character
sql_postgresql.c: In function `sql_check_error':
sql_postgresql.c:116: error: parse error before "s"
sql_postgresql.c:116:22: missing terminating " character
sql_postgresql.c:129:31: missing terminating " character
sql_postgresql.c:130: error: parse error before "s"
sql_postgresql.c:130:11: missing terminating " character
sql_postgresql.c: In function `sql_query':
sql_postgresql.c:227: warning: implicit declaration of function
`sql_store_result'
sql_postgresql.c:228: warning: implicit declaration of function
`sql_num_fields'
sql_postgresql.c: At top level:
sql_postgresql.c:257: warning: `sql_store_result' was declared
implicitly `extern' and later `static'
sql_postgresql.c:227: warning: previous declaration of
`sql_store_result'
sql_postgresql.c:274: warning: `sql_num_fields' was declared implicitly
`extern' and later `static'
sql_postgresql.c:228: warning: previous declaration of `sql_num_fields'
make[10]: *** [sql_postgresql.o] Error 1
#---


Mentioned file is:
src/modules/rlm_sql/drivers/rlm_sql_postgresql/sql_postgresql.c

Line 115 of this file is:
radlog(L_DBG, "rlm_sql_postgresql: Postgresql
check_error:
s, returning SQL_DOWN", PQresStatus(error));

(there's a newline at the end of first line.. Above code segment is
actually two lines in sql_postgresql.c)

Guess it'd be:
radlog(L_DBG, "rlm_sql_postgresql: Postgresql
check_error: %s, returning SQL_DOWN", PQresStatus(error));



The same mistake exists on line 128 too...


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