Patchwork [3/5] petitboot: Add make warnings

login
register
mail settings
Submitter Geoff Levand
Date Jan. 13, 2009, 1:29 a.m.
Message ID <496BEE78.2010409@am.sony.com>
Download mbox | patch
Permalink /patch/18132/
State Accepted
Delegated to: Jeremy Kerr
Headers show

Comments

Geoff Levand - Jan. 13, 2009, 1:29 a.m.
Add some compiler warnings to the default build.

Signed-off-by: Geoff Levand <geoffrey.levand@am.sony.com>
---
 rules.mk |    2 ++
 1 file changed, 2 insertions(+)
Jeremy Kerr - Jan. 15, 2009, 2:31 a.m.
Hi Geoff,

> +CFLAGS += -Wall -W -Wunused -Wstrict-prototypes -Wmissing-prototypes
> \ +	-Wmissing-declarations -Wredundant-decls -Winline

Probably best to do this when we initially set CFLAGS in configure.

Cheers,


Jeremy

Patch

--- a/rules.mk
+++ b/rules.mk
@@ -2,6 +2,8 @@ 
 VPATH = $(srcdir)
 
 CPPFLAGS += -I$(top_srcdir) -I$(top_srcdir)/lib -I$(builddir)
+CFLAGS += -Wall -W -Wunused -Wstrict-prototypes -Wmissing-prototypes \
+	-Wmissing-declarations -Wredundant-decls -Winline
 
 # we need paths to be overridable at build-time
 DEFS += '-DPREFIX="$(prefix)"' '-DPKG_SHARE_DIR="$(pkgdatadir)"' \