aboutsummaryrefslogtreecommitdiffstats
path: root/src/common.mk
diff options
context:
space:
mode:
authorLibravatar Kelvin M. Klann <kmk3.code@protonmail.com>2022-10-19 14:06:16 -0300
committerLibravatar Kelvin M. Klann <kmk3.code@protonmail.com>2022-11-21 17:15:29 -0300
commit60a95f00bb1a3185fce4bacb5367201685767444 (patch)
treee4f7d7ab251be37f354d8626c1c6a331c0b857b8 /src/common.mk
parentmakefiles: rename H_FILE_LIST and C_FILE_LIST (diff)
downloadfirejail-60a95f00bb1a3185fce4bacb5367201685767444.tar.gz
firejail-60a95f00bb1a3185fce4bacb5367201685767444.tar.zst
firejail-60a95f00bb1a3185fce4bacb5367201685767444.zip
makefiles: expand HDRS, SRCS and OBJS immediately
Use immediate expansion of the right-hand side (with `:=`) to set the variables to the output of the commands rather than to the (text of the) commands themselves. This should prevent deferred/lazy evaluation, which is something that might potentially result in the relevant files being looked up each time that HDRS and SRCS are evaluated. Commands used to search and replace: git grep -Ilz '^SRCS' -- src | xargs -0 -I '{}' \ sh -c "printf '%s\n' \"\$(sed \ -e 's/^HDRS =/HDRS :=/' \ -e 's/^SRCS =/SRCS :=/' \ -e 's/^OBJS =/OBJS :=/' '{}')\" >'{}'"
Diffstat (limited to 'src/common.mk')
-rw-r--r--src/common.mk6
1 files changed, 3 insertions, 3 deletions
diff --git a/src/common.mk b/src/common.mk
index 07082e183..c20aaf52f 100644
--- a/src/common.mk
+++ b/src/common.mk
@@ -2,9 +2,9 @@
2# 2#
3# Note: $(ROOT)/config.mk must be included before this file. 3# Note: $(ROOT)/config.mk must be included before this file.
4 4
5HDRS = $(sort $(wildcard *.h)) 5HDRS := $(sort $(wildcard *.h))
6SRCS = $(sort $(wildcard *.c)) 6SRCS := $(sort $(wildcard *.c))
7OBJS = $(SRCS:.c=.o) 7OBJS := $(SRCS:.c=.o)
8 8
9CFLAGS += -ggdb $(HAVE_FATAL_WARNINGS) -O2 -DVERSION='"$(VERSION)"' $(HAVE_GCOV) 9CFLAGS += -ggdb $(HAVE_FATAL_WARNINGS) -O2 -DVERSION='"$(VERSION)"' $(HAVE_GCOV)
10CFLAGS += -DPREFIX='"$(prefix)"' -DSYSCONFDIR='"$(sysconfdir)/firejail"' -DLIBDIR='"$(libdir)"' -DBINDIR='"$(bindir)"' -DVARDIR='"/var/lib/firejail"' 10CFLAGS += -DPREFIX='"$(prefix)"' -DSYSCONFDIR='"$(sysconfdir)/firejail"' -DLIBDIR='"$(libdir)"' -DBINDIR='"$(bindir)"' -DVARDIR='"/var/lib/firejail"'