From 7f5e857a11f932c247b5a96d0d86768226ba2e96 Mon Sep 17 00:00:00 2001
From: Stephen Gallagher <sgallagh@redhat.com>
Date: Mon, 13 Jul 2009 09:12:33 -0400
Subject: Build all SSSD components with warnings enabled

Previously, only the SSSD server components were being built with
compile-time warnings enabled. This patch will ensure that all
components in common and sss_client are also built the same way.
---
 common/collection/Makefile.am  | 11 ++++++++++-
 common/collection/configure.ac |  2 ++
 common/dhash/Makefile.am       |  8 +++++++-
 common/dhash/configure.ac      |  2 ++
 common/ini/Makefile.am         | 11 ++++++++++-
 common/ini/configure.ac        |  2 ++
 6 files changed, 33 insertions(+), 3 deletions(-)

(limited to 'common')

diff --git a/common/collection/Makefile.am b/common/collection/Makefile.am
index fe0749045..a802cf017 100644
--- a/common/collection/Makefile.am
+++ b/common/collection/Makefile.am
@@ -2,7 +2,16 @@
 TRACE_LEVEL=@TRACE_VAR@
 
 topdir=$(srcdir)/..
-AM_CPPFLAGS = -Wall -I$(topdir) -I$(topdir)/trace $(TRACE_LEVEL)
+
+AM_CFLAGS =
+if HAVE_GCC
+    AM_CFLAGS += \
+        -Wall -Wshadow -Wstrict-prototypes -Wpointer-arith -Wcast-qual \
+        -Wcast-align -Wwrite-strings
+endif
+
+AM_CPPFLAGS = -I$(topdir) -I$(topdir)/trace $(TRACE_LEVEL)
+
 ACLOCAL_AMFLAGS = -I m4
 
 # Set up the pkg-config file
diff --git a/common/collection/configure.ac b/common/collection/configure.ac
index 504cd4d5c..290a31017 100644
--- a/common/collection/configure.ac
+++ b/common/collection/configure.ac
@@ -7,6 +7,8 @@ AC_PROG_LIBTOOL
 AC_CONFIG_MACRO_DIR([m4])
 AC_PROG_INSTALL
 
+AM_CONDITIONAL([HAVE_GCC], [test "$ac_cv_prog_gcc" = yes])
+
 m4_pattern_allow([AM_SILENT_RULES])
 AM_SILENT_RULES
 
diff --git a/common/dhash/Makefile.am b/common/dhash/Makefile.am
index 9037bf97c..b8d5ce6ff 100644
--- a/common/dhash/Makefile.am
+++ b/common/dhash/Makefile.am
@@ -1,4 +1,10 @@
-AM_CPPFLAGS = -Wall
+AM_CFLAGS =
+if HAVE_GCC
+    AM_CFLAGS += \
+        -Wall -Wshadow -Wstrict-prototypes -Wpointer-arith -Wcast-qual \
+        -Wcast-align -Wwrite-strings
+endif
+
 ACLOCAL_AMFLAGS = -I m4
 
 pkgconfigdir = $(libdir)/pkgconfig
diff --git a/common/dhash/configure.ac b/common/dhash/configure.ac
index feabc4845..bd2102877 100644
--- a/common/dhash/configure.ac
+++ b/common/dhash/configure.ac
@@ -7,6 +7,8 @@ AC_PROG_LIBTOOL
 AC_CONFIG_MACRO_DIR([m4])
 AC_PROG_INSTALL
 
+AM_CONDITIONAL([HAVE_GCC], [test "$ac_cv_prog_gcc" = yes])
+
 m4_pattern_allow([AM_SILENT_RULES])
 AM_SILENT_RULES
 
diff --git a/common/ini/Makefile.am b/common/ini/Makefile.am
index 03d891256..dc30e9471 100644
--- a/common/ini/Makefile.am
+++ b/common/ini/Makefile.am
@@ -2,7 +2,16 @@
 TRACE_LEVEL=@TRACE_VAR@
 
 topdir=$(srcdir)/..
-AM_CPPFLAGS = -Wall -I$(topdir) -I$(topdir)/trace -I$(topdir)/collection $(TRACE_LEVEL)
+
+AM_CFLAGS =
+if HAVE_GCC
+    AM_CFLAGS += \
+        -Wall -Wshadow -Wstrict-prototypes -Wpointer-arith -Wcast-qual \
+        -Wcast-align -Wwrite-strings
+endif
+
+AM_CPPFLAGS = -I$(topdir) -I$(topdir)/trace -I$(topdir)/collection $(TRACE_LEVEL)
+
 ACLOCAL_AMFLAGS = -I m4
 
 # Set up the pkg-config file
diff --git a/common/ini/configure.ac b/common/ini/configure.ac
index 14280cfca..c4732cf79 100644
--- a/common/ini/configure.ac
+++ b/common/ini/configure.ac
@@ -7,6 +7,8 @@ AC_PROG_LIBTOOL
 AC_CONFIG_MACRO_DIR([m4])
 AC_PROG_INSTALL
 
+AM_CONDITIONAL([HAVE_GCC], [test "$ac_cv_prog_gcc" = yes])
+
 m4_pattern_allow([AM_SILENT_RULES])
 AM_SILENT_RULES
 
-- 
cgit