This replaces current code attempting to perform similar function. In configure.ac, the code is gcc specific. The src/Makefile.am has $(WARNING_FLAGS) rather than $(WARNING_CPPFLAGS)
http://cgit.freedesktop.org/xorg/util/macros/tree/xorg-macros.m4.in Looking at the xorg-gtest module --help text: --enable-strict-compilation Enable all warnings from compiler and make them errors (default: disabled) Signed-off-by: Gaetan Nadon <[email protected]> --- There are always different opinions when it comes to warnings vs errors, this patch proposes what has become a consensus in xorg over the years. configure.ac | 2 -- src/Makefile.am | 2 -- 2 files changed, 0 insertions(+), 4 deletions(-) diff --git a/configure.ac b/configure.ac index bd8e2c0..51de80f 100644 --- a/configure.ac +++ b/configure.ac @@ -57,8 +57,6 @@ AC_SUBST(DUMMY_CONF_PATH, "$datadir/xorg/gtest/dummy.conf") AC_CONFIG_FILES([Makefile xorg-gtest.pc]) -AC_SUBST(WARNING_CPPFLAGS, "-Wall -Werror") - AC_CONFIG_FILES([data/Makefile doc/Makefile examples/Makefile diff --git a/src/Makefile.am b/src/Makefile.am index a75fb9e..772e257 100644 --- a/src/Makefile.am +++ b/src/Makefile.am @@ -25,8 +25,6 @@ lib_LTLIBRARIES = libxorg-gtest.la libxorg-gtest_main.la -AM_CPPFLAGS = $(WARNING_FLAGS) - AM_CXXFLAGS = -I$(top_srcdir)/include $(XSERVER_CFLAGS) $(BASE_CXXFLAGS) libxorg_gtest_la_SOURCES = \ -- 1.7.5.4 _______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel
