Source: xpat2
Version: 1.07-20
Severity: serious
Justification: FTBFS
Tags: trixie sid ftbfs
User: lu...@debian.org
Usertags: ftbfs-20240313 ftbfs-trixie ftbfs-impfuncdef

Hi,

During a rebuild of all packages in sid, your package failed to build
on amd64.

This is most likely caused by a change in dpkg 1.22.6, that enabled
-Werror=implicit-function-declaration. For more information, see
https://wiki.debian.org/qa.debian.org/FTBFS#A2024-03-13_-Werror.3Dimplicit-function-declaration

Relevant part (hopefully):
> gcc -g -O2 -fno-strict-aliasing -g -O2 -Werror=implicit-function-declaration 
> -ffile-prefix-map=/<<PKGBUILDDIR>>=. -fstack-protector-strong 
> -fstack-clash-protection -Wformat -Werror=format-security -fcf-protection     
>    -Dlinux -D__amd64__ -D_POSIX_C_SOURCE=199309L                             
> -D_POSIX_SOURCE -D_XOPEN_SOURCE                                 -D_BSD_SOURCE 
> -D_SVID_SOURCE                                 -D_LARGEFILE_SOURCE 
> -D_FILE_OFFSET_BITS=64                                                        
>            -DFUNCPROTO=15 -DNARROWPROTO   -DLOADCARDS -DONLINE_HELP -DuseXaw  
>              -DSOUND -I/usr/include/X11 -DGFX2               
> -DSCOREFILE=\"/var/games/xpat2/xpat.log\" -DLIBDIR=\"/usr/share/games/xpat\" 
> -Wdate-time -D_FORTIFY_SOURCE=2   -c -o loadsave.o loadsave.c
> In file included from 
> /usr/include/x86_64-linux-gnu/bits/libc-header-start.h:33,
>                  from /usr/include/stdio.h:27,
>                  from xpat.h:43,
>                  from xpatgame.h:16,
>                  from loadsave.c:16:
> /usr/include/features.h:195:3: warning: #warning "_BSD_SOURCE and 
> _SVID_SOURCE are deprecated, use _DEFAULT_SOURCE" [-Wcpp]
>   195 | # warning "_BSD_SOURCE and _SVID_SOURCE are deprecated, use 
> _DEFAULT_SOURCE"
>       |   ^~~~~~~
> loadsave.c: In function ‘write_log_file’:
> loadsave.c:279:24: error: implicit declaration of function ‘cuserid’ 
> [-Werror=implicit-function-declaration]
>   279 |         || (username = cuserid(NULL)) != NULL
>       |                        ^~~~~~~
> loadsave.c:279:22: warning: assignment to ‘const char *’ from ‘int’ makes 
> pointer from integer without a cast [-Wint-conversion]
>   279 |         || (username = cuserid(NULL)) != NULL
>       |                      ^
> loadsave.c: In function ‘switch_uid’:
> loadsave.c:55:9: warning: ignoring return value of ‘setuid’ declared with 
> attribute ‘warn_unused_result’ [-Wunused-result]
>    55 |         setuid(to_real ? real_uid : effective_uid);
>       |         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> loadsave.c:56:9: warning: ignoring return value of ‘setgid’ declared with 
> attribute ‘warn_unused_result’ [-Wunused-result]
>    56 |         setgid(to_real ? real_gid : effective_gid);
>       |         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> cc1: some warnings being treated as errors
> make[2]: *** [<builtin>: loadsave.o] Error 1


The full build log is available from:
http://qa-logs.debian.net/2024/03/13/xpat2_1.07-20_unstable.log

All bugs filed during this archive rebuild are listed at:
https://bugs.debian.org/cgi-bin/pkgreport.cgi?tag=ftbfs-20240313;users=lu...@debian.org
or:
https://udd.debian.org/bugs/?release=na&merged=ign&fnewerval=7&flastmodval=7&fusertag=only&fusertagtag=ftbfs-20240313&fusertaguser=lu...@debian.org&allbugs=1&cseverity=1&ctags=1&caffected=1#results

A list of current common problems and possible solutions is available at
http://wiki.debian.org/qa.debian.org/FTBFS . You're welcome to contribute!

If you reassign this bug to another package, please mark it as 'affects'-ing
this package. See https://www.debian.org/Bugs/server-control#affects

If you fail to reproduce this, please provide a build log and diff it with mine
so that we can identify if something relevant changed in the meantime.

Reply via email to