FS#9297 - libalpm simple compile error with --enable-debug

Attached to Project: Pacman
Opened by Vesa Kaihlavirta (vegai) - Tuesday, 22 January 2008, 08:47 GMT
Last edited by Xavier (shining) - Sunday, 27 January 2008, 11:11 GMT
Task Type Bug Report
Category Backend/Core
Status Closed
Assigned To Dan McGee (toofishes)
Architecture x86_64
Severity Medium
Priority Normal
Reported Version git
Due in Version 3.2.0
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Configured with --enable-debug.

$ make
make all-recursive
make[1]: Entering directory `/home/vegai/git/pacman'
Making all in lib/libalpm
make[2]: Entering directory `/home/vegai/git/pacman/lib/libalpm'
Making all in po
make[3]: Entering directory `/home/vegai/git/pacman/lib/libalpm/po'
make[3]: Nothing to be done for `all'.
make[3]: Leaving directory `/home/vegai/git/pacman/lib/libalpm/po'
make[3]: Entering directory `/home/vegai/git/pacman/lib/libalpm'
/bin/sh ../../libtool --tag=CC --mode=compile gcc -std=gnu99 -DLOCALEDIR=\"/home/vegai/pacman//share/locale\" -DHAVE_CONFIG_H -I. -I../.. -pedantic -D_GNU_SOURCE -fvisibility=internal -fgnu89-inline -g -O2 -fstack-protector-all -Wall -Werror -MT deps.lo -MD -MP -MF .deps/deps.Tpo -c -o deps.lo deps.c
gcc -std=gnu99 -DLOCALEDIR=\"/home/vegai/pacman//share/locale\" -DHAVE_CONFIG_H -I. -I../.. -pedantic -D_GNU_SOURCE -fvisibility=internal -fgnu89-inline -g -O2 -fstack-protector-all -Wall -Werror -MT deps.lo -MD -MP -MF .deps/deps.Tpo -c deps.c -fPIC -DPIC -o .libs/deps.o
cc1: warnings being treated as errors
deps.c: In function '_alpm_splitdep':
deps.c:403: warning: format '%zd' expects type 'signed size_t', but argument 3 has type 'int'
deps.c: In function '_alpm_dep_dup':
deps.c:448: warning: format '%zd' expects type 'signed size_t', but argument 3 has type 'int'
make[3]: *** [deps.lo] Error 1
make[3]: Leaving directory `/home/vegai/git/pacman/lib/libalpm'
make[2]: *** [all-recursive] Error 1
make[2]: Leaving directory `/home/vegai/git/pacman/lib/libalpm'
make[1]: *** [all-recursive] Error 1
make[1]: Leaving directory `/home/vegai/git/pacman'
make: *** [all] Error 2


$ git diff lib/libalpm/deps.c > CALLOC_cast_literal_to_sizet.patch

...attached.
   CALLOC_cast_literal_to_sizet.... (1 KiB)
This task depends upon

Closed by  Xavier (shining)
Sunday, 27 January 2008, 11:11 GMT
Reason for closing:  Fixed
Additional comments about closing:  Fixed by commit 69c6d59bb6
Comment by Xavier (shining) - Tuesday, 22 January 2008, 17:54 GMT
Hmm, I can't reproduce this. So I am just curious :
Are you building this on arch (testing or not?)? And no changes to the toolchain?
Comment by Xavier (shining) - Tuesday, 22 January 2008, 17:59 GMT
vegai just told me he is on x86_64, so that might explain it :)
Comment by Vesa Kaihlavirta (vegai) - Tuesday, 22 January 2008, 18:00 GMT
As we discussed on IRC, this seems to happen only on x86_64. Makes sense if you think about size_t...
Comment by Dan McGee (toofishes) - Wednesday, 23 January 2008, 00:51 GMT
  • Field changed: Status (New → Assigned)
  • Field changed: Reported Version (3.1.1 → git)
  • Task assigned to Dan McGee (toofishes)
How about this instead? Let's fix it correctly instead of dirty casts- we just had the arguments flipped here.
   correctfix.patch (1 KiB)
Comment by Vesa Kaihlavirta (vegai) - Wednesday, 23 January 2008, 06:38 GMT
Indeed. That's much better.

Loading...