build: Combine $(CFLAGS) and $(STRICT_CFLAGS).
authorAndre Noll <maan@tuebingen.mpg.de>
Sun, 17 Jul 2016 16:27:13 +0000 (18:27 +0200)
committerAndre Noll <maan@tuebingen.mpg.de>
Sun, 26 Mar 2017 09:02:28 +0000 (11:02 +0200)
commit16e6f52a4a9bba6ea98c1c06abfb28582bab9184
tree981ae95b48c66be55e529f6e4198c1661e0c8b94
parent9059ae879362cbab03c5c0892143799ca881e47b
build: Combine $(CFLAGS) and $(STRICT_CFLAGS).

We used to compile the parsers generated by gengetopt with less strict
flags to avoid warnings in generated files. With lopsub this is no
longer necessary, so one set of flags suffices.

Moreover, we currently initialize $(STRICT_CFLAGS) to $(CFLAGS) (from
the environment), then append our own flags. This is unfortunate
because the user defined settings in $(CFLAGS) are overwritten.

This patch changes the Makefile to only set STRICT_CFLAGS. We now
pass $(STRICT_CFLAGS) $(CFLAGS) to the compiler, allowing the user
to override (parts of) $(STRICT_CFLAGS) by adding -Wno-foo to CFLAGS.

We use the opportunity to regroup the various options a little.
Makefile.real