From e665769f5225062565b3a75677ce43eb37b988c5 Mon Sep 17 00:00:00 2001 From: "Kelvin M. Klann" Date: Wed, 5 Jul 2023 23:56:06 -0300 Subject: build: automatically generate header dependencies Instead of manually specifying which source files depend on which headers, use compiler flags to automatically generate depfiles (.d), which declare the correct header (make) dependencies for each source file (.c). Use `-MMD` (which ignores system headers) to generate the dependencies and `-MP` to prevent make from complaining when a header file is removed while it is listed as a dependency in a depfile. If depfiles exist, just include them. If not, make each object file (.o) unconditionally depend on all header files in its source directory and in src/include, to ensure that rebuilds are done when needed. The latter case applies to the first build after `make clean` (which would build everything anyway) and when the compiler does not support generating depfiles. Note that both gcc and clang have supported these options for a long time. Misc: This depends on the changes from commit 5b1bd33c7 ("build: use full paths on compile/link targets", 2023-07-02) / PR #6158 to avoid issues with make dependency tracking. --- src/etc-cleanup/Makefile | 2 -- src/fbuilder/Makefile | 2 -- src/fcopy/Makefile | 1 - src/fids/Makefile | 2 -- src/firecfg/Makefile | 7 ------- src/firejail/Makefile | 13 ------------- src/firemon/Makefile | 1 - src/fldd/Makefile | 1 - src/fnet/Makefile | 1 - src/fnetfilter/Makefile | 1 - src/fsec-optimize/Makefile | 1 - src/fsec-print/Makefile | 1 - src/fseccomp/Makefile | 1 - src/fzenity/Makefile | 2 -- src/jailcheck/Makefile | 1 - src/libpostexecseccomp/Makefile | 2 -- src/libtracelog/Makefile | 2 -- src/profstats/Makefile | 2 -- src/prog.mk | 12 +++++++++--- src/so.mk | 12 +++++++++--- 20 files changed, 18 insertions(+), 49 deletions(-) (limited to 'src') diff --git a/src/etc-cleanup/Makefile b/src/etc-cleanup/Makefile index 23121c521..d9dcdbea8 100644 --- a/src/etc-cleanup/Makefile +++ b/src/etc-cleanup/Makefile @@ -7,6 +7,4 @@ MOD_DIR = $(ROOT)/src/$(MOD) PROG = $(MOD_DIR)/$(MOD) TARGET = $(PROG) -EXTRA_HDRS = ../include/etc_groups.h - include $(ROOT)/src/prog.mk diff --git a/src/fbuilder/Makefile b/src/fbuilder/Makefile index f0e374a52..70c5fb008 100644 --- a/src/fbuilder/Makefile +++ b/src/fbuilder/Makefile @@ -7,6 +7,4 @@ MOD_DIR = $(ROOT)/src/$(MOD) PROG = $(MOD_DIR)/$(MOD) TARGET = $(PROG) -EXTRA_HDRS = ../include/common.h ../include/syscall.h - include $(ROOT)/src/prog.mk diff --git a/src/fcopy/Makefile b/src/fcopy/Makefile index a9f706eb8..4897980e7 100644 --- a/src/fcopy/Makefile +++ b/src/fcopy/Makefile @@ -7,7 +7,6 @@ MOD_DIR = $(ROOT)/src/$(MOD) PROG = $(MOD_DIR)/$(MOD) TARGET = $(PROG) -EXTRA_HDRS = ../include/common.h ../include/syscall.h EXTRA_OBJS = ../lib/common.o include $(ROOT)/src/prog.mk diff --git a/src/fids/Makefile b/src/fids/Makefile index 7bdbe0590..dadb17a1b 100644 --- a/src/fids/Makefile +++ b/src/fids/Makefile @@ -7,6 +7,4 @@ MOD_DIR = $(ROOT)/src/$(MOD) PROG = $(MOD_DIR)/$(MOD) TARGET = $(PROG) -EXTRA_HDRS = ../include/common.h - include $(ROOT)/src/prog.mk diff --git a/src/firecfg/Makefile b/src/firecfg/Makefile index cc7a27c52..6cbd1b5ba 100644 --- a/src/firecfg/Makefile +++ b/src/firecfg/Makefile @@ -7,13 +7,6 @@ MOD_DIR = $(ROOT)/src/$(MOD) PROG = $(MOD_DIR)/$(MOD) TARGET = $(PROG) -EXTRA_HDRS = \ -../include/common.h \ -../include/euid_common.h \ -../include/libnetlink.h \ -../include/firejail_user.h \ -../include/pid.h - EXTRA_OBJS = ../lib/common.o ../lib/firejail_user.o include $(ROOT)/src/prog.mk diff --git a/src/firejail/Makefile b/src/firejail/Makefile index 1c4b1b5c2..e7b334cd8 100644 --- a/src/firejail/Makefile +++ b/src/firejail/Makefile @@ -7,19 +7,6 @@ MOD_DIR = $(ROOT)/src/$(MOD) PROG = $(MOD_DIR)/$(MOD) TARGET = $(PROG) -EXTRA_HDRS = \ -../include/rundefs.h \ -../include/common.h \ -../include/ldd_utils.h \ -../include/euid_common.h \ -../include/pid.h \ -../include/seccomp.h \ -../include/syscall_i386.h \ -../include/syscall_x86_64.h \ -../include/firejail_user.h \ -../include/etc_groups.h - - EXTRA_OBJS = \ ../lib/common.o \ ../lib/ldd_utils.o \ diff --git a/src/firemon/Makefile b/src/firemon/Makefile index 0e39f8b01..02e3b6eed 100644 --- a/src/firemon/Makefile +++ b/src/firemon/Makefile @@ -7,7 +7,6 @@ MOD_DIR = $(ROOT)/src/$(MOD) PROG = $(MOD_DIR)/$(MOD) TARGET = $(PROG) -EXTRA_HDRS = ../include/common.h ../include/pid.h EXTRA_OBJS = ../lib/common.o ../lib/pid.o include $(ROOT)/src/prog.mk diff --git a/src/fldd/Makefile b/src/fldd/Makefile index 246cbc183..3e48b43ab 100644 --- a/src/fldd/Makefile +++ b/src/fldd/Makefile @@ -7,7 +7,6 @@ MOD_DIR = $(ROOT)/src/$(MOD) PROG = $(MOD_DIR)/$(MOD) TARGET = $(PROG) -EXTRA_HDRS = ../include/common.h ../include/syscall.h ../include/ldd_utils.h EXTRA_OBJS = ../lib/common.o ../lib/ldd_utils.o include $(ROOT)/src/prog.mk diff --git a/src/fnet/Makefile b/src/fnet/Makefile index c98326dc4..dfeefdc47 100644 --- a/src/fnet/Makefile +++ b/src/fnet/Makefile @@ -7,7 +7,6 @@ MOD_DIR = $(ROOT)/src/$(MOD) PROG = $(MOD_DIR)/$(MOD) TARGET = $(PROG) -EXTRA_HDRS = ../include/common.h ../include/libnetlink.h EXTRA_OBJS = ../lib/common.o ../lib/libnetlink.o include $(ROOT)/src/prog.mk diff --git a/src/fnetfilter/Makefile b/src/fnetfilter/Makefile index c8992c3d6..778de3a8b 100644 --- a/src/fnetfilter/Makefile +++ b/src/fnetfilter/Makefile @@ -7,7 +7,6 @@ MOD_DIR = $(ROOT)/src/$(MOD) PROG = $(MOD_DIR)/$(MOD) TARGET = $(PROG) -EXTRA_HDRS = ../include/common.h ../include/syscall.h EXTRA_OBJS = ../lib/common.o include $(ROOT)/src/prog.mk diff --git a/src/fsec-optimize/Makefile b/src/fsec-optimize/Makefile index caead9d54..babed1fdb 100644 --- a/src/fsec-optimize/Makefile +++ b/src/fsec-optimize/Makefile @@ -7,7 +7,6 @@ MOD_DIR = $(ROOT)/src/$(MOD) PROG = $(MOD_DIR)/$(MOD) TARGET = $(PROG) -EXTRA_HDRS = ../include/common.h ../include/seccomp.h ../include/syscall.h EXTRA_OBJS = ../lib/common.o ../lib/errno.o include $(ROOT)/src/prog.mk diff --git a/src/fsec-print/Makefile b/src/fsec-print/Makefile index 43cc42fed..bf383d2d0 100644 --- a/src/fsec-print/Makefile +++ b/src/fsec-print/Makefile @@ -7,7 +7,6 @@ MOD_DIR = $(ROOT)/src/$(MOD) PROG = $(MOD_DIR)/$(MOD) TARGET = $(PROG) -EXTRA_HDRS = ../include/common.h ../include/seccomp.h ../include/syscall.h EXTRA_OBJS = ../lib/common.o ../lib/errno.o ../lib/syscall.o include $(ROOT)/src/prog.mk diff --git a/src/fseccomp/Makefile b/src/fseccomp/Makefile index 89b12f629..66bb58444 100644 --- a/src/fseccomp/Makefile +++ b/src/fseccomp/Makefile @@ -7,7 +7,6 @@ MOD_DIR = $(ROOT)/src/$(MOD) PROG = $(MOD_DIR)/$(MOD) TARGET = $(PROG) -EXTRA_HDRS = ../include/common.h ../include/syscall.h EXTRA_OBJS = ../lib/common.o ../lib/errno.o ../lib/syscall.o include $(ROOT)/src/prog.mk diff --git a/src/fzenity/Makefile b/src/fzenity/Makefile index 3c1aa0574..4cebe7ac5 100644 --- a/src/fzenity/Makefile +++ b/src/fzenity/Makefile @@ -7,6 +7,4 @@ MOD_DIR = $(ROOT)/src/$(MOD) PROG = $(MOD_DIR)/$(MOD) TARGET = $(PROG) -EXTRA_HDRS = ../include/common.h - include $(ROOT)/src/prog.mk diff --git a/src/jailcheck/Makefile b/src/jailcheck/Makefile index 1823ee9d0..431a21c8e 100644 --- a/src/jailcheck/Makefile +++ b/src/jailcheck/Makefile @@ -7,7 +7,6 @@ MOD_DIR = $(ROOT)/src/$(MOD) PROG = $(MOD_DIR)/$(MOD) TARGET = $(PROG) -EXTRA_HDRS = ../include/common.h ../include/pid.h EXTRA_OBJS = ../lib/common.o ../lib/pid.o include $(ROOT)/src/prog.mk diff --git a/src/libpostexecseccomp/Makefile b/src/libpostexecseccomp/Makefile index f66fdd0d7..a8d3fd8b7 100644 --- a/src/libpostexecseccomp/Makefile +++ b/src/libpostexecseccomp/Makefile @@ -7,6 +7,4 @@ MOD_DIR = $(ROOT)/src/$(MOD) SO = $(MOD_DIR)/$(MOD).so TARGET = $(SO) -EXTRA_HDRS = ../include/seccomp.h ../include/rundefs.h - include $(ROOT)/src/so.mk diff --git a/src/libtracelog/Makefile b/src/libtracelog/Makefile index a83726167..e9330236c 100644 --- a/src/libtracelog/Makefile +++ b/src/libtracelog/Makefile @@ -7,6 +7,4 @@ MOD_DIR = $(ROOT)/src/$(MOD) SO = $(MOD_DIR)/$(MOD).so TARGET = $(SO) -EXTRA_HDRS = ../include/rundefs.h - include $(ROOT)/src/so.mk diff --git a/src/profstats/Makefile b/src/profstats/Makefile index 8f33ce1de..213b1476d 100644 --- a/src/profstats/Makefile +++ b/src/profstats/Makefile @@ -7,6 +7,4 @@ MOD_DIR = $(ROOT)/src/$(MOD) PROG = $(MOD_DIR)/$(MOD) TARGET = $(PROG) -EXTRA_HDRS = ../include/common.h - include $(ROOT)/src/prog.mk diff --git a/src/prog.mk b/src/prog.mk index debdf3572..a639e87fc 100644 --- a/src/prog.mk +++ b/src/prog.mk @@ -3,16 +3,22 @@ # Note: $(ROOT)/config.mk must be included before this file. # # The includer should probably define PROG and TARGET and may also want to -# define EXTRA_HDRS and EXTRA_OBJS and extend CLEANFILES. +# define EXTRA_OBJS and extend CLEANFILES. -HDRS := $(sort $(wildcard $(MOD_DIR)/*.h)) +HDRS := SRCS := $(sort $(wildcard $(MOD_DIR)/*.c)) OBJS := $(SRCS:.c=.o) +DEPS := $(sort $(wildcard $(OBJS:.o=.d))) + +ifeq ($(DEPS),) +HDRS := $(sort $(wildcard $(MOD_DIR)/*.h $(ROOT)/src/include/*.h)) +endif .PHONY: all all: $(TARGET) +-include $(DEPS) -%.o : %.c $(HDRS) $(EXTRA_HDRS) $(ROOT)/config.mk +%.o : %.c $(HDRS) $(ROOT)/config.mk $(CC) $(PROG_CFLAGS) $(CFLAGS) $(CPPFLAGS) -c $< -o $@ $(PROG): $(OBJS) $(EXTRA_OBJS) $(ROOT)/config.mk diff --git a/src/so.mk b/src/so.mk index 95b524081..ac76ffc30 100644 --- a/src/so.mk +++ b/src/so.mk @@ -3,16 +3,22 @@ # Note: $(ROOT)/config.mk must be included before this file. # # The includer should probably define SO and TARGET and may also want to define -# EXTRA_HDRS and EXTRA_OBJS and extend CLEANFILES. +# EXTRA_OBJS and extend CLEANFILES. -HDRS := $(sort $(wildcard $(MOD_DIR)/*.h)) +HDRS := SRCS := $(sort $(wildcard $(MOD_DIR)/*.c)) OBJS := $(SRCS:.c=.o) +DEPS := $(sort $(wildcard $(OBJS:.o=.d))) + +ifeq ($(DEPS),) +HDRS := $(sort $(wildcard $(MOD_DIR)/*.h $(ROOT)/src/include/*.h)) +endif .PHONY: all all: $(TARGET) +-include $(DEPS) -%.o : %.c $(HDRS) $(EXTRA_HDRS) $(ROOT)/config.mk +%.o : %.c $(HDRS) $(ROOT)/config.mk $(CC) $(SO_CFLAGS) $(CFLAGS) $(CPPFLAGS) -c $< -o $@ $(SO): $(OBJS) $(EXTRA_OBJS) $(ROOT)/config.mk -- cgit v1.2.3-54-g00ecf