[Pkg-crosswire-devel] Bug#811602: FTBFS with GCC 6: multiple errors

[Martin Michlmayr]

Package: sword
Version: 1.7.3+dfsg-7
Severity: important
User: debian-gcc at lists.debian.org
Usertags: ftbfs-gcc-6 gcc-6-misleading-indentation gcc-6-address

This package fails to build with GCC 6.  GCC 6 has not been released
yet, but it's expected that GCC 6 will become the default compiler for
stretch.

Note that only the first error is reported; there might be more.  You
can find a snapshot of GCC 6 in experimental.  To build with GCC 6,
you can set CC=gcc-6 CXX=g++-6 explicitly.

> sbuild (Debian sbuild) 0.67.0 (26 Dec 2015) on dl580gen9-02.hlinux
...
> /usr/bin/c++   -DCLUCENE2 -DCURLAVAILABLE -DCURLSFTPAVAILABLE -DGLOBCONFPATH=\"/etc/sword.conf\" -DUSELUCENE -D_FTPLIB_NO_COMPAT -D_ICU_ -I/<<BUILDDIR>>/sword-1.7.3+dfsg/obj-x86_64-linux-gnu/include -I/<<BUILDDIR>>/sword-1.7.3+dfsg/include -I/usr/lib/x86_64-linux-gnu -I/usr/include/x86_64-linux-gnu  -g -O2 -fstack-protector-strong -Wformat -Werror=format-security -Wall -Werror -Wno-error=deprecated-declarations -Wdate-time -D_FORTIFY_SOURCE=2    -fPIC -o CMakeFiles/sword_static.dir/src/mgr/remotetrans.cpp.o -c /<<BUILDDIR>>/sword-1.7.3+dfsg/src/mgr/remotetrans.cpp
> /usr/bin/c++   -DCLUCENE2 -DCURLAVAILABLE -DCURLSFTPAVAILABLE -DGLOBCONFPATH=\"/etc/sword.conf\" -DUSELUCENE -D_FTPLIB_NO_COMPAT -D_ICU_ -Dsword_EXPORTS -I/<<BUILDDIR>>/sword-1.7.3+dfsg/obj-x86_64-linux-gnu/include -I/<<BUILDDIR>>/sword-1.7.3+dfsg/include -I/usr/lib/x86_64-linux-gnu -I/usr/include/x86_64-linux-gnu  -g -O2 -fstack-protector-strong -Wformat -Werror=format-security -Wall -Werror -Wno-error=deprecated-declarations -Wdate-time -D_FORTIFY_SOURCE=2  -fPIC   -o CMakeFiles/sword.dir/src/mgr/encfiltmgr.cpp.o -c /<<BUILDDIR>>/sword-1.7.3+dfsg/src/mgr/encfiltmgr.cpp
> /<<BUILDDIR>>/sword-1.7.3+dfsg/src/keys/treekeyidx.cpp: In member function 'virtual void sword::TreeKeyIdx::remove()':
> /<<BUILDDIR>>/sword-1.7.3+dfsg/src/keys/treekeyidx.cpp:264:7: error: statement is indented as if it were guarded by... [-Werror=misleading-indentation]
>        if (iterator.next > -1) {
>        ^~
> 
> /<<BUILDDIR>>/sword-1.7.3+dfsg/src/keys/treekeyidx.cpp:262:6: note: ...this 'while' clause, but it is not
>       while ((iterator.next != target) && (iterator.next > -1))
>       ^~~~~
> 
> /<<BUILDDIR>>/sword-1.7.3+dfsg/src/keys/treekeyidx.cpp: In member function 'virtual void sword::TreeKeyIdx::remove()':
> /<<BUILDDIR>>/sword-1.7.3+dfsg/src/keys/treekeyidx.cpp:264:7: error: statement is indented as if it were guarded by... [-Werror=misleading-indentation]
>        if (iterator.next > -1) {
>        ^~
> 
> /<<BUILDDIR>>/sword-1.7.3+dfsg/src/keys/treekeyidx.cpp:262:6: note: ...this 'while' clause, but it is not
>       while ((iterator.next != target) && (iterator.next > -1))
>       ^~~~~
> 
> In file included from /<<BUILDDIR>>/sword-1.7.3+dfsg/include/swkey.h:28:0,
>                  from /<<BUILDDIR>>/sword-1.7.3+dfsg/include/treekey.h:27,
>                  from /<<BUILDDIR>>/sword-1.7.3+dfsg/include/treekeyidx.h:27,
>                  from /<<BUILDDIR>>/sword-1.7.3+dfsg/src/keys/treekeyidx.cpp:24:
> /<<BUILDDIR>>/sword-1.7.3+dfsg/src/keys/treekeyidx.cpp: In member function 'virtual int sword::TreeKeyIdx::compare(const sword::SWKey&)':
> /<<BUILDDIR>>/sword-1.7.3+dfsg/include/swobject.h:30:128: error: the compiler can assume that the address of 'ikey' will always evaluate to 'true' [-Werror=address]
>  #define SWDYNAMIC_CAST(className, object) (className *)((object)?((object->getClass()->isAssignableFrom(#className))?object:0):0)
>                                                                                                                                 ^
> 
> /<<BUILDDIR>>/sword-1.7.3+dfsg/src/keys/treekeyidx.cpp:575:24: note: in expansion of macro 'SWDYNAMIC_CAST'
>   TreeKeyIdx *treeKey = SWDYNAMIC_CAST(TreeKeyIdx, (&ikey));
>                         ^~~~~~~~~~~~~~
> 
> In file included from /<<BUILDDIR>>/sword-1.7.3+dfsg/include/swkey.h:28:0,
>                  from /<<BUILDDIR>>/sword-1.7.3+dfsg/include/treekey.h:27,
>                  from /<<BUILDDIR>>/sword-1.7.3+dfsg/include/treekeyidx.h:27,
>                  from /<<BUILDDIR>>/sword-1.7.3+dfsg/src/keys/treekeyidx.cpp:24:
> /<<BUILDDIR>>/sword-1.7.3+dfsg/src/keys/treekeyidx.cpp: In member function 'virtual int sword::TreeKeyIdx::compare(const sword::SWKey&)':
> /<<BUILDDIR>>/sword-1.7.3+dfsg/include/swobject.h:30:128: error: the compiler can assume that the address of 'ikey' will always evaluate to 'true' [-Werror=address]
>  #define SWDYNAMIC_CAST(className, object) (className *)((object)?((object->getClass()->isAssignableFrom(#className))?object:0):0)
>                                                                                                                                 ^
> 
> /<<BUILDDIR>>/sword-1.7.3+dfsg/src/keys/treekeyidx.cpp:575:24: note: in expansion of macro 'SWDYNAMIC_CAST'
>   TreeKeyIdx *treeKey = SWDYNAMIC_CAST(TreeKeyIdx, (&ikey));
>                         ^~~~~~~~~~~~~~
> 

-- 
Martin Michlmayr
Linux for HPE Helion, Hewlett Packard Enterprise