[PATCH 2/2] Let package manager override CFLAGS and CPPFLAGS

Jes Sorensen
2020-05-15T20:57:03Z

Package managers such as RPM wants to build everything with their
preferred flags, and we shouldn't hard override flags.

Signed-off-by: Jes Sorensen <jsorensen@fb.com>
---
Makefile | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/Makefile b/Makefile
index c5f46f4..0c2a621 100644
--- a/Makefile
+++ b/Makefile
@@ -32,7 +32,7 @@ cc-option = $(shell if $(CC) $(1) -c -x c /dev/null -o /dev/null &>/dev/null; \
#### Common compiler flags. You can add additional flags by defining CFLAGS
#### and/or CPPFLAGS in the environment or on the 'make' command line.

-override CFLAGS := -O2 -Wall -Wundef \
+CFLAGS := -O2 -Wall -Wundef \
$(call cc-option,-Wdeclaration-after-statement) \
$(call cc-option,-Wmissing-prototypes) \
$(call cc-option,-Wstrict-prototypes) \
@@ -40,7 +40,7 @@ override CFLAGS := -O2 -Wall -Wundef \
$(call cc-option,-Wimplicit-fallthrough) \
$(CFLAGS)

-override CPPFLAGS := -D_FILE_OFFSET_BITS=64 $(CPPFLAGS)
+CPPFLAGS := -D_FILE_OFFSET_BITS=64 $(CPPFLAGS)

#### Other user settings

--
2.26.2