On Sun, Mar 07, 2021 at 08:30:13PM +0100, Jerome KASPER wrote:
> Hi klemmens,
>
> First of all thanks for testing and reviewing.
> Nice spots in the grammar description.
> I will poke upstream about adding CFLAGS +=
> in the config.mk and will report back.
> I didn't knew about CPPFLAGS overriding with
> CFLAGS, is it a clang trick for debug?
No, that is just how this port's config.mk works:
$ tail $(make show=WRKSRC)/config.mk
INCS = -I${LOCALINC} -I${X11INC} -I${FREETYPEINC}
LIBS = -L${LOCALLIB} -L${X11LIB} -lm -lfontconfig -lXft -lX11
-lXinerama -lXrender -lXext -lImlib2
# flags
CPPFLAGS =
CFLAGS = -Wall -Wextra ${INCS} ${CPPFLAGS}
LDFLAGS = ${LIBS}
# compiler and linker
CC = cc
So I simply overwrite the empty string via command line such that our
CFLAGS end up at the end of the port's CFLAGS.
Hence I wouldn't do this if config.mk actually used non-empty CPPFLAGS.
