From 47c2aa0b71cabac1b50752a146cb2d8d4e0f304e Mon Sep 17 00:00:00 2001 From: Mike Frysinger Date: Thu, 12 Mar 2015 03:56:06 -0400 Subject: split -Werror out into a dedicated var This lets us control it explicitly. In CrOS, we won't turn it off, but most distros will want to as it's too hard to keep a handle on compilers and various settings users leverage. BUG=chromium:466499 TEST=precq still passes BRANCH=None Change-Id: I11bf03acefa3fd624dd4c5688dd18e3a3c87647d Reviewed-on: https://chromium-review.googlesource.com/259521 Reviewed-by: Randall Spangler Tested-by: Mike Frysinger Commit-Queue: Mike Frysinger --- Makefile | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/Makefile b/Makefile index a9abdbbc..5645a8cd 100644 --- a/Makefile +++ b/Makefile @@ -120,9 +120,10 @@ endif # # Flag ordering: arch, then -f, then -m, then -W DEBUG_FLAGS := $(if ${DEBUG},-g -O0,-Os) +WERROR := -Werror COMMON_FLAGS := -nostdinc -pipe \ -ffreestanding -fno-builtin -fno-stack-protector \ - -Werror -Wall -Wstrict-prototypes ${DEBUG_FLAGS} + ${WERROR} -Wall -Wstrict-prototypes ${DEBUG_FLAGS} # Note: FIRMWARE_ARCH is defined by the Chromium OS ebuild. ifeq (${FIRMWARE_ARCH}, arm) -- cgit v1.2.1