From: Uwe Kleine-König <u.kleine-koenig@xxxxxxxxxxxxxx> For compilation to work -D_GNU_SOURCE -Isrc/include is needed to be passed to the compiler. For Debian packaging several things are added to CFLAGS but not these two from above yielding compilation errors, similar to: $ make CFLAGS=-Wall cyclictest cc -D VERSION_STRING=0.84 -c src/cyclictest/cyclictest.c -Wall In file included from src/cyclictest/cyclictest.c:37:0: src/cyclictest/rt_numa.h:17:22: fatal error: rt-utils.h: No such file or directory compilation terminated. make: *** [cyclictest.o] Error 1 So add the needed parameters unconditionally. There is no harm if they are included in the user supplied CFLAGS and so passed twice. Moreover be a bit more correct about CFLAGS/CPPFLAGS. Both should be passed to the compiler with CFLAGS taking options for the compiler and CPPFLAGS taking options for the preprocessor. This is also needed for Debian packaging where the helper scripts set CPPFLAGS. Signed-off-by: Uwe Kleine-König <u.kleine-koenig@xxxxxxxxxxxxxx> --- Makefile | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) --- a/Makefile +++ b/Makefile @@ -20,7 +20,8 @@ ifneq ($(filter x86_64 i386 ia64 mips po NUMA := 1 endif -CFLAGS ?= -D_GNU_SOURCE -Wall -Wno-nonnull -Isrc/include +CFLAGS ?= -Wall -Wno-nonnull +CPPFLAGS += -D_GNU_SOURCE -Isrc/include LDFLAGS ?= ifndef DEBUG @@ -47,11 +48,11 @@ VPATH += src/lib VPATH += src/hackbench %.o: %.c - $(CC) -D VERSION_STRING=$(VERSION_STRING) -c $< $(CFLAGS) + $(CC) -D VERSION_STRING=$(VERSION_STRING) -c $< $(CFLAGS) $(CPPFLAGS) # Pattern rule to generate dependency files from .c files %.d: %.c - @$(CC) -MM $(CFLAGS) $< | sed 's,\($*\)\.o[ :]*,\1.o $@ : ,g' > $@ || rm -f $@ + @$(CC) -MM $(CFLAGS) $(CPPFLAGS) $< | sed 's,\($*\)\.o[ :]*,\1.o $@ : ,g' > $@ || rm -f $@ .PHONY: all all: $(TARGETS) hwlatdetect -- To unsubscribe from this list: send the line "unsubscribe linux-rt-users" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
