[PATCH libdrm 4/4] autotools: add WARN_CFLAGS to all targets
Emil Velikov
emil.l.velikov at gmail.com
Mon Feb 23 05:57:42 PST 2015
... minus test/ttmtest. The latter is not really hooked up with the
actual build.
This will give us 66 warnings on a distribution build of which
- 12 -Wunused-variable
- 11 -Wunused-function
- 19 -Wmissing-prototypes
and a few -Wswitch-enum, -Wtype-limits etc.
Adding the CFLAGS gives some exposure to these so that we can fix them.
Signed-off-by: Emil Velikov <emil.l.velikov at gmail.com>
---
Makefile.am | 1 +
tests/Makefile.am | 3 ++-
tests/exynos/Makefile.am | 1 +
tests/kmstest/Makefile.am | 1 +
tests/modeprint/Makefile.am | 1 +
tests/proptest/Makefile.am | 1 +
tests/radeon/Makefile.am | 1 +
tests/tegra/Makefile.am | 2 +-
tests/vbltest/Makefile.am | 2 ++
9 files changed, 11 insertions(+), 2 deletions(-)
diff --git a/Makefile.am b/Makefile.am
index 062feb4..9514cc5 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -82,6 +82,7 @@ libdrm_la_LIBADD = @CLOCK_LIB@
libdrm_la_CPPFLAGS = -I$(top_srcdir)/include/drm
AM_CFLAGS = \
+ $(WARN_CFLAGS) \
$(VALGRIND_CFLAGS)
libdrm_la_SOURCES = $(LIBDRM_FILES)
diff --git a/tests/Makefile.am b/tests/Makefile.am
index 37b8d3a..f989d8e 100644
--- a/tests/Makefile.am
+++ b/tests/Makefile.am
@@ -1,6 +1,7 @@
NULL:=#
-AM_CPPFLAGS = \
+AM_CFLAGS = \
+ $(WARN_CFLAGS)\
-I $(top_srcdir)/include/drm \
-I $(top_srcdir)
diff --git a/tests/exynos/Makefile.am b/tests/exynos/Makefile.am
index 92de4e4..b21d016 100644
--- a/tests/exynos/Makefile.am
+++ b/tests/exynos/Makefile.am
@@ -1,4 +1,5 @@
AM_CFLAGS = \
+ $(WARN_CFLAGS)\
-I $(top_srcdir)/include/drm \
-I $(top_srcdir)/libkms/ \
-I $(top_srcdir)/exynos \
diff --git a/tests/kmstest/Makefile.am b/tests/kmstest/Makefile.am
index 7903a26..fd21e61 100644
--- a/tests/kmstest/Makefile.am
+++ b/tests/kmstest/Makefile.am
@@ -1,4 +1,5 @@
AM_CFLAGS = \
+ $(WARN_CFLAGS)\
-I$(top_srcdir)/include/drm \
-I$(top_srcdir)/libkms/ \
-I$(top_srcdir)
diff --git a/tests/modeprint/Makefile.am b/tests/modeprint/Makefile.am
index 6420ef3..895805f 100644
--- a/tests/modeprint/Makefile.am
+++ b/tests/modeprint/Makefile.am
@@ -1,4 +1,5 @@
AM_CFLAGS = \
+ $(WARN_CFLAGS)\
-I$(top_srcdir)/include/drm \
-I$(top_srcdir)
diff --git a/tests/proptest/Makefile.am b/tests/proptest/Makefile.am
index f81a3c0..48a84c1 100644
--- a/tests/proptest/Makefile.am
+++ b/tests/proptest/Makefile.am
@@ -1,4 +1,5 @@
AM_CFLAGS = \
+ $(WARN_CFLAGS)\
-I$(top_srcdir)/include/drm \
-I$(top_srcdir)
diff --git a/tests/radeon/Makefile.am b/tests/radeon/Makefile.am
index 1775669..d18620d 100644
--- a/tests/radeon/Makefile.am
+++ b/tests/radeon/Makefile.am
@@ -1,4 +1,5 @@
AM_CFLAGS = \
+ $(WARN_CFLAGS)\
-I $(top_srcdir)/include/drm \
-I $(top_srcdir)
diff --git a/tests/tegra/Makefile.am b/tests/tegra/Makefile.am
index ca63d92..8e625c8 100644
--- a/tests/tegra/Makefile.am
+++ b/tests/tegra/Makefile.am
@@ -3,7 +3,7 @@ AM_CPPFLAGS = \
-I$(top_srcdir)/tegra \
-I$(top_srcdir)
-AM_CFLAGS = -Wall -Werror
+AM_CFLAGS = $(WARN_CFLAGS)
LDADD = \
../../tegra/libdrm_tegra.la \
diff --git a/tests/vbltest/Makefile.am b/tests/vbltest/Makefile.am
index 34a35e7..4d87887 100644
--- a/tests/vbltest/Makefile.am
+++ b/tests/vbltest/Makefile.am
@@ -1,6 +1,8 @@
AM_CFLAGS = \
+ $(WARN_CFLAGS)\
-I$(top_srcdir)/include/drm \
-I$(top_srcdir)
+
if HAVE_INSTALL_TESTS
bin_PROGRAMS = \
vbltest
--
2.3.0
More information about the dri-devel
mailing list