[PATCH] Makefile: add $(DEVELOPER_CFLAGS) variable
- From
Matthieu Moy <matthieu.moy@imag.fr>
- Date
- Jun 1, 2016, 08:00 UTC
- Message-ID
- <20160601080008.7348-1-Matthieu.Moy@imag.fr>
- In-Reply-To
- <vpqpos11gv3.fsf@anie.imag.fr>
This does not change the behavior, but allows the user to tweak DEVELOPER_CFLAGS on the command-line or in a config.mak* file if needed.
This also makes the code somewhat cleaner as it follows the pattern
<initialisation of variables> <include statements> <actual build logic>
by specifying which flags to activate in the first part, and actually activating them in the last one.
Signed-off-by: Matthieu Moy <Matthieu.Moy@imag.fr> --- Junio, you can add this to mm/makefile-developer-can-be-in-config-mak (or squash it in the commit, but having two separate commit messages make sense IMO).
Makefile | 19 ++++++++++--------- 1 file changed, 10 insertions(+), 9 deletions(-)
diff --git a/Makefile b/Makefile index 2226319..9753abe 100644 --- a/Makefile +++ b/Makefile @@ -375,6 +375,15 @@ GIT-VERSION-FILE: FORCE # CFLAGS and LDFLAGS are for the users to override from the command line. CFLAGS = -g -O2 -Wall +DEVELOPER_CFLAGS = -Werror \ + -Wdeclaration-after-statement \ + -Wno-format-zero-length \ + -Wold-style-definition \ + -Woverflow \ + -Wpointer-arith \ + -Wstrict-prototypes \ + -Wunused \ + -Wvla LDFLAGS = ALL_CFLAGS = $(CPPFLAGS) $(CFLAGS) ALL_LDFLAGS = $(LDFLAGS) @@ -941,15 +950,7 @@ include config.mak.uname -include config.mak ifdef DEVELOPER -CFLAGS += -Werror \ - -Wdeclaration-after-statement \ - -Wno-format-zero-length \ - -Wold-style-definition \ - -Woverflow \ - -Wpointer-arith \ - -Wstrict-prototypes \ - -Wunused \ - -Wvla +CFLAGS += $(DEVELOPER_CFLAGS) endif ifndef sysconfdir
-- 2.8.2.397.gbe91ebf.dirty