You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Jens Rehsack <re...@googlemail.com> on 2009/04/30 18:01:35 UTC

Subversion 1.6.1 broken on AIX compiling with xlc

Hi all,

I assist Greg updating the subversion-package for PkgSrc and tried out
whether it builds fine on AIX - it doesn't, but the problem was really small.

> cc -I/opt/pkgsrc/include -I/usr/include -I/opt/pkgsrc/include/apr-1 
> -U__STR__ -D_THREAD_SAFE -D_USE_IRS  -q64 -qarch=ppc64 -O2 -qstrict 
> -qcpluscmt -qmaxmem=-1 -qkeyword=inline -I/opt/pkgsrc/include -ma 
> -I/usr/include -I/opt/pkgsrc/include/apr-1   -qHALT=E 
> -I./subversion/include -I./subversion  -I/opt/pkgsrc/include/apr-1 
> -I/opt/pkgsrc/include/apr-1 -I/opt/pkgsrc/include 
> -I/opt/pkgsrc/include/neon -o subversion/svnserve/serve.o -c 
> subversion/svnserve/serve.c
> "subversion/svnserve/serve.c", line 1981.21: 1506-343 (S) Redeclaration of 
> stat differs from previous declaration on line 320 of 
> "/usr/include/sys/stat.h".
> "subversion/svnserve/serve.c", line 1981.21: 1506-050 (I) Return type 
> "struct svn_error_t*" in redeclaration is not compatible with the previous 
> return type "int".
> "subversion/svnserve/serve.c", line 1981.21: 1506-376 (I) Redeclaration of 
> stat has a different number of fixed parameters than the previous 
> declaration.
> "subversion/svnserve/serve.c", line 1981.21: 1506-377 (I) The type "struct 
> svn_ra_svn_conn_st*" of parameter 1 differs from the previous type "const 
> char* restrict".

It seems that the xlc doesn't like the redefinition of the stat() function.
My solution was easy - I renamed the stat() function to stat_cmd() - see
attached patch.

Best regards,
Jens

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=1998824

Re: Subversion 1.6.1 broken on AIX compiling with xlc

Posted by Eric Gillespie <ep...@pretzelnet.org>.
Jens Rehsack <re...@googlemail.com> writes:

> It seems that the xlc doesn't like the redefinition of the stat() function.
> My solution was easy - I renamed the stat() function to stat_cmd() - see
> attached patch.

+1

I've done this before when building svnserve with dietlibc.

-- 
Eric Gillespie <*> epg@pretzelnet.org

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=2019343