Revert "Makefile: Fix compilation after header removal."
authorAndre Noll <maan@tuebingen.mpg.de>
Mon, 30 May 2022 19:06:32 +0000 (21:06 +0200)
committerAndre Noll <maan@tuebingen.mpg.de>
Mon, 30 May 2022 19:06:32 +0000 (21:06 +0200)
This reverts commit 3bc858ee0d9b929f4cbca61cf5ed8d3184d0cf78.

It is broken because the recipe for the .d files does not honor
CPPFLAGS since we only modify the CPPFLAGS variable for the .o
targets. Adding the corresponding .d targets would be too ugly,
so revert the commit for now until a better solution has been found.

Makefile.real

index e5ecfe4..6e8084d 100644 (file)
@@ -255,19 +255,10 @@ $(object_dir)/mm.o \
 
 $(object_dir)/compress_filter.o: CFLAGS += -O3
 
 
 $(object_dir)/compress_filter.o: CFLAGS += -O3
 
-define CC_CMD
+$(object_dir)/%.o: %.c | $(object_dir) $(dep_dir) $(lsg_h) $(yy_h)
        $(call SAY, CC $<)
        $(call SAY, CC $<)
-       $(CC) -c -o $(object_dir)/$(*F).o -MMD -MF \
-               $(dep_dir)/$(*F).d -MT $(object_dir)/$(*F).o \
-               $(CPPFLAGS) $(STRICT_CFLAGS) $(CFLAGS) $<
-endef
-CC_PREREQUISITES := %.c | $(object_dir) $(dep_dir) $(lsg_h) $(yy_h)
-# These two have the same prerequisites and the same recipe. There should be a
-# better way to write this.
-$(object_dir)/%.o: $(CC_PREREQUISITES)
-       $(CC_CMD)
-$(dep_dir)/%.d: $(CC_PREREQUISITES)
-       $(CC_CMD)
+       $(CC) -c -o $@ -MMD -MF $(dep_dir)/$(*F).d -MT $@ $(CPPFLAGS) \
+               $(STRICT_CFLAGS) $(CFLAGS) $<
 
 para_recv para_afh para_play para_server: LDFLAGS += $(id3tag_ldflags)
 para_write para_play para_audiod \
 
 para_recv para_afh para_play para_server: LDFLAGS += $(id3tag_ldflags)
 para_write para_play para_audiod \