Menu

#21 libxklavier should set -Wall and -Werror in configure file

open
nobody
None
5
2005-05-13
2005-05-13
Anonymous
No

libxklavier currently doesn't build with compilers
other than gcc unless you hack the Makefile.am files to
remove -Wall and -Werror. Most modules add these to
CFLAGS in configure.in via a technique like this:

AC_MSG_CHECKING(for more warnings, including -Werror)
if test "$GCC" = "yes" -a "$set_more_warnings" != "no";
then
AC_MSG_RESULT(yes)
CFLAGS="\ -Wall \ -Wchar-subscripts -Wmissing-declarations
-Wmissing-prototypes \ -Wnested-externs -Wpointer-arith \ -Wcast-align -Wsign-compare \ -std=gnu89 \ $CFLAGS"
else
AC_MSG_RESULT(no)
fi

Could this be used in libxklavier for better non-gcc
compiler support?

Discussion

  • Nobody/Anonymous

    Logged In: NO

    Oh, this was reported by Brian.Cameron@sun.com

     
  • Damien Carbery

    Damien Carbery - 2005-05-25

    Logged In: YES
    user_id=843868

    As a workaround, I simply removed "-Wall -Werror" from
    tests/Makefile.am and libxklavier/Makefile.am so I could
    build from cvs head on Solaris 10, using the forte compiler.

     

Log in to post a comment.