Patchwork [4/21] acpid: modify CFLAGS

login
register
mail settings
Submitter Mark Hatle
Date May 29, 2013, 3:09 p.m.
Message ID <1369840203-21898-5-git-send-email-mark.hatle@windriver.com>
Download mbox | patch
Permalink /patch/50711/
State New
Headers show

Comments

Mark Hatle - May 29, 2013, 3:09 p.m.
From: Joe Slater <jslater@windriver.com>

Override the hard-coded CFLAGS used in Makefile to reference our CFLAGS.

Without this patch if the DEBUG_OPTIMIZATION is enabled (using -O0) the
compile log shows acpid still using -O2 because the Makefile has various
hard coded CFLAGS defined.  Instead of using the hard coded CFLAGS,
we simply define the proper set within the recipe itself.

Signed-off-by: Joe Slater <jslater@windriver.com>
Signed-off-by: Jeff Polk <jeff.polk@windriver.com>
Signed-off-by: Mark Hatle <mark.hatle@windriver.com>
---
 meta/recipes-bsp/acpid/acpid.inc | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

Patch

diff --git a/meta/recipes-bsp/acpid/acpid.inc b/meta/recipes-bsp/acpid/acpid.inc
index 3c0219f..1ffe6ff 100644
--- a/meta/recipes-bsp/acpid/acpid.inc
+++ b/meta/recipes-bsp/acpid/acpid.inc
@@ -13,7 +13,9 @@  inherit update-rc.d
 INITSCRIPT_NAME = "acpid"
 INITSCRIPT_PARAMS = "defaults"
 
-EXTRA_OEMAKE = ""
+# Makefile ignores our CFLAGS, so override it.
+#
+EXTRA_OEMAKE = "CFLAGS='-W -Wall -Werror -Wundef -Wshadow ${CFLAGS} $(DEFS)'"
 
 do_compile () {
 	oe_runmake 'CC=${CC} -D_GNU_SOURCE' 'CROSS=${HOST_PREFIX}'