Bug#984361: tcm: ftbfs with GCC-11
Package: src:tcm
Version: 2.20+TSQD-6
Severity: normal
Tags: sid bookworm
User: debian-gcc@lists.debian.org
Usertags: ftbfs-gcc-11
[This bug is not targeted to the upcoming bullseye release]
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-11/g++-11, but succeeds to build with gcc-10/g++-10. The
severity of this report will be raised before the bookworm release,
so nothing has to be done for the bullseye release.
The full build log can be found at:
http://people.debian.org/~doko/logs/20210228/filtered/gcc11/tcm_2.20+TSQD-6_unstable_gcc11.log
The last lines of the build log are at the end of this report.
To build with GCC 11, either set CC=gcc-11 CXX=g++-11 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-11/porting_to.html
GCC 11 defaults to the GNU++17 standard. If your package installs
header files in /usr/include, please don't work around C++17 issues
by choosing a lower C++ standard for the package build, but fix these
issues to build with the C++17 standard.
[...]
from modejunction.h:25,
from modejunction.c:22:
/<<PKGBUILDDIR>>/src/gl/llist.h: In member function ‘int List<T>::next()’:
/<<PKGBUILDDIR>>/src/gl/llist.h:106:17: warning: this ‘if’ clause does not guard... [-Wmisleading-indentation]
106 | if (current) current=current->next; return (current != 0); }
| ^~
/<<PKGBUILDDIR>>/src/gl/llist.h:106:53: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the ‘if’
106 | if (current) current=current->next; return (current != 0); }
| ^~~~~~
/<<PKGBUILDDIR>>/src/gl/llist.h: In member function ‘int List<T>::prev()’:
/<<PKGBUILDDIR>>/src/gl/llist.h:110:17: warning: this ‘if’ clause does not guard... [-Wmisleading-indentation]
110 | if (current) current=current->prev; return (current != 0); }
| ^~
/<<PKGBUILDDIR>>/src/gl/llist.h:110:53: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the ‘if’
110 | if (current) current=current->prev; return (current != 0); }
| ^~~~~~
/usr/bin/g++ -Wall -g -fcommon -O2 -DCONFIG_INSTALL=\"/etc/tcm/\" -DTCM_INSTALL_DIR=\"/usr\" -DTCM_INSTALL_LIB=\"/usr/lib/\" -DTCM_INSTALL_SHARE=\"/usr/share/doc/tcm-doc/\" -DCONFIG_FILE=\"tcm.conf\" -DHELP_DIR=\"/usr/share/doc/tcm-doc/help/\" -DCOLOR_FILE=\"colorrgb.txt\" -DBANNER_FILE=\"banner.ps\" -I. -I/<<PKGBUILDDIR>>/src/bitmaps -I/<<PKGBUILDDIR>>/src/gl -I/<<PKGBUILDDIR>>/src/ui -I/<<PKGBUILDDIR>>/src/ed -I/<<PKGBUILDDIR>>/src/dg -I/<<PKGBUILDDIR>>/src/tb -I/usr/X11R6/LessTif/Motif1.2/include -I/usr/X11R6/include -I/usr/X11R6/include/X11 -I../dv -DLINUX -c cbactor.c
In file included from /<<PKGBUILDDIR>>/src/dg/thing.h:25,
from /<<PKGBUILDDIR>>/src/dg/subject.h:25,
from /<<PKGBUILDDIR>>/src/dg/node.h:25,
from cbactor.h:26,
from cbactor.c:23:
/<<PKGBUILDDIR>>/src/gl/llist.h: In member function ‘int List<T>::next()’:
/<<PKGBUILDDIR>>/src/gl/llist.h:106:17: warning: this ‘if’ clause does not guard... [-Wmisleading-indentation]
106 | if (current) current=current->next; return (current != 0); }
| ^~
/<<PKGBUILDDIR>>/src/gl/llist.h:106:53: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the ‘if’
106 | if (current) current=current->next; return (current != 0); }
| ^~~~~~
/<<PKGBUILDDIR>>/src/gl/llist.h: In member function ‘int List<T>::prev()’:
/<<PKGBUILDDIR>>/src/gl/llist.h:110:17: warning: this ‘if’ clause does not guard... [-Wmisleading-indentation]
110 | if (current) current=current->prev; return (current != 0); }
| ^~
/<<PKGBUILDDIR>>/src/gl/llist.h:110:53: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the ‘if’
110 | if (current) current=current->prev; return (current != 0); }
| ^~~~~~
/usr/bin/g++ -Wall -g -fcommon -O2 -DCONFIG_INSTALL=\"/etc/tcm/\" -DTCM_INSTALL_DIR=\"/usr\" -DTCM_INSTALL_LIB=\"/usr/lib/\" -DTCM_INSTALL_SHARE=\"/usr/share/doc/tcm-doc/\" -DCONFIG_FILE=\"tcm.conf\" -DHELP_DIR=\"/usr/share/doc/tcm-doc/help/\" -DCOLOR_FILE=\"colorrgb.txt\" -DBANNER_FILE=\"banner.ps\" -I. -I/<<PKGBUILDDIR>>/src/bitmaps -I/<<PKGBUILDDIR>>/src/gl -I/<<PKGBUILDDIR>>/src/ui -I/<<PKGBUILDDIR>>/src/ed -I/<<PKGBUILDDIR>>/src/dg -I/<<PKGBUILDDIR>>/src/tb -I/usr/X11R6/LessTif/Motif1.2/include -I/usr/X11R6/include -I/usr/X11R6/include/X11 -I../dv -DLINUX -c c2r2mlistline.c
In file included from /<<PKGBUILDDIR>>/src/ui/grafport.h:30,
from c2r2mlistline.c:25:
/<<PKGBUILDDIR>>/src/gl/llist.h: In member function ‘int List<T>::next()’:
/<<PKGBUILDDIR>>/src/gl/llist.h:106:17: warning: this ‘if’ clause does not guard... [-Wmisleading-indentation]
106 | if (current) current=current->next; return (current != 0); }
| ^~
/<<PKGBUILDDIR>>/src/gl/llist.h:106:53: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the ‘if’
106 | if (current) current=current->next; return (current != 0); }
| ^~~~~~
/<<PKGBUILDDIR>>/src/gl/llist.h: In member function ‘int List<T>::prev()’:
/<<PKGBUILDDIR>>/src/gl/llist.h:110:17: warning: this ‘if’ clause does not guard... [-Wmisleading-indentation]
110 | if (current) current=current->prev; return (current != 0); }
| ^~
/<<PKGBUILDDIR>>/src/gl/llist.h:110:53: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the ‘if’
110 | if (current) current=current->prev; return (current != 0); }
| ^~~~~~
c2r2mlistline.c: In member function ‘virtual void C2R2MListLine::DrawShape()’:
c2r2mlistline.c:423:13: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
423 | if ((CBDMessage *)((CBObjectLinkEdge *)GetSubject())->NrOfMessages() > 0) {
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
c2r2mlistline.c:423:78: error: ordered comparison of pointer with integer zero (‘CBDMessage*’ and ‘int’)
423 | if ((CBDMessage *)((CBObjectLinkEdge *)GetSubject())->NrOfMessages() > 0) {
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~
make[4]: *** [/<<PKGBUILDDIR>>/src/Config.GEN:62: c2r2mlistline.o] Error 1
make[4]: Leaving directory '/<<PKGBUILDDIR>>/src/sd/dv'
make[3]: *** [Makefile:6: tcmd] Error 2
make[3]: Leaving directory '/<<PKGBUILDDIR>>/src/sd'
make[2]: *** [Makefile:29: tcmd] Error 2
make[2]: Leaving directory '/<<PKGBUILDDIR>>/src'
make[1]: *** [Makefile:113: execs] Error 2
make[1]: Leaving directory '/<<PKGBUILDDIR>>'
make: *** [debian/rules:29: build-stamp] Error 2
dpkg-buildpackage: error: debian/rules build subprocess returned exit status 2
Reply to: