hi, newest build of mtr cannot be compiled with gcc-3.4.6 (current version on hardened). it does not know -Wno-pointer-sign option. if i686-pc-linux-gnu-gcc -DHAVE_CONFIG_H -I. -I. -I. -march=pentium2 -O3 -pipe -Wall -Wno-pointer-sign -MT mtr.o -MD -MP -MF ".deps/mtr.Tpo" -c -o mtr.o mtr.c; \ then mv -f ".deps/mtr.Tpo" ".deps/mtr.Po"; else rm -f ".deps/mtr.Tpo"; exit 1; fi cc1: error: unrecognized command line option "-Wno-pointer-sign" make[2]: *** [mtr.o] Error 1 fix: new ebuild
This can be fixed with a simple configure patch such as: { echo "$as_me:$LINENO: checking for C flags to get more warnings" >&5 echo $ECHO_N "checking for C flags to get more warnings... $ECHO_C" >&6; } ac_save_CFLAGS="$CFLAGS" if test "x$ac_cv_c_compiler_gnu" = "xyes" ; then - warning_CFLAGS="-Wall -Wno-pointer-sign" + if test "`gcc -dumpversion`" = "3.4.6" ; then + warning_CFLAGS="-Wall" + else + warning_CFLAGS="-Wall -Wno-pointer-sign" + fi; else case "$host_os" in irix*)tes In that path, it's already been determined that a GNU toolchain is in use, so presumably calling gcc -dumpversion is OK. It's flawed though because it should really detect any version of GNU GCC lower than that which introduced the (no)-pointer-sign warning option and not set it on that basis. Unfortunately, while I can think of a number of ways to do that, I don't know autotools well enough to propose a method that is well suited to a configure script.
Created attachment 192346 [details, diff] mtr-0.75--Wno-pointer-sign.patch Thank you for report, barthek. I have bad memory so I'll attach patch for this issue here. I've reported it upstream and I want to give them some time to answer before I push this patch to the tree.
Ok, upstream ack'ed patch. commited. Thank you for report.
