Your message dated Sat, 5 Aug 2017 00:35:29 +0200
with message-id <20170804223529.donhvk5ba4w72...@aurel32.net>
and subject line Bug#853587: binutils-source: does not build with 
-Werror=pointer-compare
has caused the Debian Bug report #853587,
regarding openbios: ftbfs with GCC-7
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact ow...@bugs.debian.org
immediately.)


-- 
853587: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=853587
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems
--- Begin Message ---
Package: src:openbios
Version: 1.1.git20161120-1
Severity: normal
Tags: sid buster
User: debian-...@lists.debian.org
Usertags: ftbfs-gcc-7

Please keep this issue open in the bug tracker for the package it
was filed for.  If a fix in another package is required, please
file a bug for the other package (or clone), and add a block in this
package. Please keep the issue open until the package can be built in
a follow-up test rebuild.

The package fails to build in a test rebuild on at least amd64 with
gcc-7/g++-7, but succeeds to build with gcc-6/g++-6. The
severity of this report may be raised before the buster release.
There is no need to fix this issue in time for the stretch release.

The full build log can be found at:
http://people.debian.org/~doko/logs/gcc7-20170126/openbios_1.1.git20161120-1_unstable_gcc7.log
The last lines of the build log are at the end of this report.

To build with GCC 7, either set CC=gcc-7 CXX=g++-7 explicitly,
or install the gcc, g++, gfortran, ... packages from experimental.

  apt-get -t=experimental install g++ 

Common build failures are new warnings resulting in build failures with
-Werror turned on, or new/dropped symbols in Debian symbols files.
For other C/C++ related build failures see the porting guide at
http://gcc.gnu.org/gcc-7/porting_to.html

[...]
/bin/bash ./libtool --tag=CC   --mode=link x86_64-linux-gnu-gcc -W -Wall 
-Wstrict-prototypes -Wmissing-prototypes -Wshadow -Wstack-usage=262144 -Werror 
-I/<<PKGBUILDDIR>>/cross-toolchain/binutils-source/binutils/../zlib -g -O2  
-static-libstdc++ -static-libgcc  -o elfedit elfedit.o version.o elfcomm.o  
../libiberty/libiberty.a 
libtool: link: x86_64-linux-gnu-gcc -W -Wall -Wstrict-prototypes 
-Wmissing-prototypes -Wshadow -Wstack-usage=262144 -Werror 
-I/<<PKGBUILDDIR>>/cross-toolchain/binutils-source/binutils/../zlib -g -O2 
-static-libstdc++ -static-libgcc -o bfdtest1 bfdtest1.o  ../bfd/.libs/libbfd.a 
-L/<<PKGBUILDDIR>>/cross-toolchain/binutils-powerpc-elf/zlib -lz 
../libiberty/libiberty.a
mv -f .deps/addr2line.Tpo .deps/addr2line.Po
/bin/bash ./libtool --tag=CC   --mode=link x86_64-linux-gnu-gcc -W -Wall 
-Wstrict-prototypes -Wmissing-prototypes -Wshadow -Wstack-usage=262144 -Werror 
-I/<<PKGBUILDDIR>>/cross-toolchain/binutils-source/binutils/../zlib -g -O2  
-static-libstdc++ -static-libgcc  -o addr2line addr2line.o bucomm.o version.o 
filemode.o ../bfd/libbfd.la ../libiberty/libiberty.a  
mv -f .deps/arlex.Tpo .deps/arlex.Po
mv -f .deps/unwind-ia64.Tpo .deps/unwind-ia64.Po
libtool: link: x86_64-linux-gnu-gcc -W -Wall -Wstrict-prototypes 
-Wmissing-prototypes -Wshadow -Wstack-usage=262144 -Werror 
-I/<<PKGBUILDDIR>>/cross-toolchain/binutils-source/binutils/../zlib -g -O2 
-static-libstdc++ -static-libgcc -o size size.o bucomm.o version.o filemode.o  
../bfd/.libs/libbfd.a 
-L/<<PKGBUILDDIR>>/cross-toolchain/binutils-powerpc-elf/zlib -lz 
../libiberty/libiberty.a
libtool: link: x86_64-linux-gnu-gcc -W -Wall -Wstrict-prototypes 
-Wmissing-prototypes -Wshadow -Wstack-usage=262144 -Werror 
-I/<<PKGBUILDDIR>>/cross-toolchain/binutils-source/binutils/../zlib -g -O2 
-static-libstdc++ -static-libgcc -o cxxfilt cxxfilt.o bucomm.o version.o 
filemode.o  ../bfd/.libs/libbfd.a 
-L/<<PKGBUILDDIR>>/cross-toolchain/binutils-powerpc-elf/zlib -lz 
../libiberty/libiberty.a
libtool: link: x86_64-linux-gnu-gcc -W -Wall -Wstrict-prototypes 
-Wmissing-prototypes -Wshadow -Wstack-usage=262144 -Werror 
-I/<<PKGBUILDDIR>>/cross-toolchain/binutils-source/binutils/../zlib -g -O2 
-static-libstdc++ -static-libgcc -o elfedit elfedit.o version.o elfcomm.o  
../libiberty/libiberty.a
libtool: link: x86_64-linux-gnu-gcc -W -Wall -Wstrict-prototypes 
-Wmissing-prototypes -Wshadow -Wstack-usage=262144 -Werror 
-I/<<PKGBUILDDIR>>/cross-toolchain/binutils-source/binutils/../zlib -g -O2 
-static-libstdc++ -static-libgcc -o strings strings.o bucomm.o version.o 
filemode.o  ../bfd/.libs/libbfd.a 
-L/<<PKGBUILDDIR>>/cross-toolchain/binutils-powerpc-elf/zlib -lz 
../libiberty/libiberty.a
mv -f .deps/ar.Tpo .deps/ar.Po
/bin/bash ./libtool --tag=CC   --mode=link x86_64-linux-gnu-gcc -W -Wall 
-Wstrict-prototypes -Wmissing-prototypes -Wshadow -Wstack-usage=262144 -Werror 
-I/<<PKGBUILDDIR>>/cross-toolchain/binutils-source/binutils/../zlib -g -O2  
-static-libstdc++ -static-libgcc  -o ranlib ar.o is-ranlib.o arparse.o arlex.o 
arsup.o rename.o binemul.o emul_vanilla.o bucomm.o version.o filemode.o 
../bfd/libbfd.la ../libiberty/libiberty.a   
/bin/bash ./libtool --tag=CC   --mode=link x86_64-linux-gnu-gcc -W -Wall 
-Wstrict-prototypes -Wmissing-prototypes -Wshadow -Wstack-usage=262144 -Werror 
-I/<<PKGBUILDDIR>>/cross-toolchain/binutils-source/binutils/../zlib -g -O2  
-static-libstdc++ -static-libgcc  -o ar arparse.o arlex.o ar.o not-ranlib.o 
arsup.o rename.o binemul.o emul_vanilla.o bucomm.o version.o filemode.o 
../bfd/libbfd.la ../libiberty/libiberty.a   
libtool: link: x86_64-linux-gnu-gcc -W -Wall -Wstrict-prototypes 
-Wmissing-prototypes -Wshadow -Wstack-usage=262144 -Werror 
-I/<<PKGBUILDDIR>>/cross-toolchain/binutils-source/binutils/../zlib -g -O2 
-static-libstdc++ -static-libgcc -o addr2line addr2line.o bucomm.o version.o 
filemode.o  ../bfd/.libs/libbfd.a 
-L/<<PKGBUILDDIR>>/cross-toolchain/binutils-powerpc-elf/zlib -lz 
../libiberty/libiberty.a
libtool: link: x86_64-linux-gnu-gcc -W -Wall -Wstrict-prototypes 
-Wmissing-prototypes -Wshadow -Wstack-usage=262144 -Werror 
-I/<<PKGBUILDDIR>>/cross-toolchain/binutils-source/binutils/../zlib -g -O2 
-static-libstdc++ -static-libgcc -o ar arparse.o arlex.o ar.o not-ranlib.o 
arsup.o rename.o binemul.o emul_vanilla.o bucomm.o version.o filemode.o  
../bfd/.libs/libbfd.a 
-L/<<PKGBUILDDIR>>/cross-toolchain/binutils-powerpc-elf/zlib -lz 
../libiberty/libiberty.a
libtool: link: x86_64-linux-gnu-gcc -W -Wall -Wstrict-prototypes 
-Wmissing-prototypes -Wshadow -Wstack-usage=262144 -Werror 
-I/<<PKGBUILDDIR>>/cross-toolchain/binutils-source/binutils/../zlib -g -O2 
-static-libstdc++ -static-libgcc -o ranlib ar.o is-ranlib.o arparse.o arlex.o 
arsup.o rename.o binemul.o emul_vanilla.o bucomm.o version.o filemode.o  
../bfd/.libs/libbfd.a 
-L/<<PKGBUILDDIR>>/cross-toolchain/binutils-powerpc-elf/zlib -lz 
../libiberty/libiberty.a
mv -f .deps/debug.Tpo .deps/debug.Po
mv -f .deps/nm.Tpo .deps/nm.Po
/bin/bash ./libtool --tag=CC   --mode=link x86_64-linux-gnu-gcc -W -Wall 
-Wstrict-prototypes -Wmissing-prototypes -Wshadow -Wstack-usage=262144 -Werror 
-I/<<PKGBUILDDIR>>/cross-toolchain/binutils-source/binutils/../zlib -g -O2  
-static-libstdc++ -static-libgcc  -o nm-new nm.o bucomm.o version.o filemode.o 
../bfd/libbfd.la ../libiberty/libiberty.a  
mv -f .deps/prdbg.Tpo .deps/prdbg.Po
mv -f .deps/wrstabs.Tpo .deps/wrstabs.Po
libtool: link: x86_64-linux-gnu-gcc -W -Wall -Wstrict-prototypes 
-Wmissing-prototypes -Wshadow -Wstack-usage=262144 -Werror 
-I/<<PKGBUILDDIR>>/cross-toolchain/binutils-source/binutils/../zlib -g -O2 
-static-libstdc++ -static-libgcc -o nm-new nm.o bucomm.o version.o filemode.o  
../bfd/.libs/libbfd.a 
-L/<<PKGBUILDDIR>>/cross-toolchain/binutils-powerpc-elf/zlib -lz 
../libiberty/libiberty.a
cc1: all warnings being treated as errors
mv -f .deps/objdump.Tpo .deps/objdump.Po
mv -f .deps/objcopy.Tpo .deps/objcopy.Po
Makefile:962: recipe for target 'stabs.o' failed
make[6]: *** [stabs.o] Error 1
make[6]: *** Waiting for unfinished jobs....
mv -f .deps/ieee.Tpo .deps/ieee.Po
mv -f .deps/dwarf.Tpo .deps/dwarf.Po
mv -f .deps/readelf.Tpo .deps/readelf.Po
make[6]: Leaving directory 
'/<<PKGBUILDDIR>>/cross-toolchain/binutils-powerpc-elf/binutils'
Makefile:1004: recipe for target 'all-recursive' failed
make[5]: *** [all-recursive] Error 1
make[5]: Leaving directory 
'/<<PKGBUILDDIR>>/cross-toolchain/binutils-powerpc-elf/binutils'
Makefile:663: recipe for target 'all' failed
make[4]: *** [all] Error 2
make[4]: Leaving directory 
'/<<PKGBUILDDIR>>/cross-toolchain/binutils-powerpc-elf/binutils'
Makefile:3609: recipe for target 'all-binutils' failed
make[3]: *** [all-binutils] Error 2
make[3]: Leaving directory 
'/<<PKGBUILDDIR>>/cross-toolchain/binutils-powerpc-elf'
Makefile:849: recipe for target 'all' failed
make[2]: *** [all] Error 2
make[2]: Leaving directory 
'/<<PKGBUILDDIR>>/cross-toolchain/binutils-powerpc-elf'
debian/cross-toolchain.mk:47: recipe for target 
'/<<PKGBUILDDIR>>/cross-toolchain/stamp-binutils_powerpc-elf' failed
make[1]: *** [/<<PKGBUILDDIR>>/cross-toolchain/stamp-binutils_powerpc-elf] 
Error 2
make[1]: Leaving directory '/<<PKGBUILDDIR>>'
debian/rules:26: recipe for target 'build' failed
make: *** [build] Error 2
dpkg-buildpackage: error: debian/rules build gave error exit status 2

--- End Message ---
--- Begin Message ---
Version: 2.19-1

On 2017-04-19 15:09, Aurelien Jarno wrote:
> clone 853587 -1
> reassign -1 binutils
> retitle -1 binutils-source: does not build with -Werror=pointer-compare
> block 853587 by -1
> thanks
> 
> On 2017-01-31 09:34, Matthias Klose wrote:
> > Package: src:openbios
> > Version: 1.1.git20161120-1
> > Severity: normal
> > Tags: sid buster
> > User: debian-...@lists.debian.org
> > Usertags: ftbfs-gcc-7
> > 
> > Please keep this issue open in the bug tracker for the package it
> > was filed for.  If a fix in another package is required, please
> > file a bug for the other package (or clone), and add a block in this
> > package. Please keep the issue open until the package can be built in
> > a follow-up test rebuild.
> > 
> > The package fails to build in a test rebuild on at least amd64 with
> > gcc-7/g++-7, but succeeds to build with gcc-6/g++-6. The
> > severity of this report may be raised before the buster release.
> > There is no need to fix this issue in time for the stretch release.
> > 
> > The full build log can be found at:
> > http://people.debian.org/~doko/logs/gcc7-20170126/openbios_1.1.git20161120-1_unstable_gcc7.log
> > The last lines of the build log are at the end of this report.
> > 
> > To build with GCC 7, either set CC=gcc-7 CXX=g++-7 explicitly,
> > or install the gcc, g++, gfortran, ... packages from experimental.
> > 
> >   apt-get -t=experimental install g++ 
> > 
> > Common build failures are new warnings resulting in build failures with
> > -Werror turned on, or new/dropped symbols in Debian symbols files.
> > For other C/C++ related build failures see the porting guide at
> > http://gcc.gnu.org/gcc-7/porting_to.html
> 
> The issue comes from binutils-source, which does not build with 
> does not build when using -Werror=pointer-compare.
> 
> | /<<PKGBUILDDIR>>/cross-toolchain/binutils-source/binutils/stabs.c: In 
> function 'parse_stab_members':
> | /<<PKGBUILDDIR>>/cross-toolchain/binutils-source/binutils/stabs.c:2705:31: 
> error: comparison between pointer and zero character constant 
> [-Werror=pointer-compare]
> |         if (**pp == ';' || *pp == '\0')
> |                                ^~
> | /<<PKGBUILDDIR>>/cross-toolchain/binutils-source/binutils/stabs.c:2705:27: 
> note: did you mean to dereference the pointer?
> |         if (**pp == ';' || *pp == '\0')
> |                            ^
> 
> The issue has been fixed upstream in the following commit:
> 
> | commit b972a0d61a8b6e1dc3f9baf93ef3e57c7bfb9715
> | Author: Nick Clifton <ni...@redhat.com>
> | Date:   Tue Jan 10 11:45:12 2017 +0000
> | 
> |     Fix compile time warning about pointer comparison.
> |     
> |             PR 21034
> |             * stabs.c (parse_stab_members): Fix thinko checking for g++
> |             version 1 stabs information.
> 

This commit is included in binutils 2.19. Closing the bug.

-- 
Aurelien Jarno                          GPG: 4096R/1DDD8C9B
aurel...@aurel32.net                 http://www.aurel32.net

--- End Message ---

Reply via email to