diff options
author | Kelvin M. Klann <kmk3.code@protonmail.com> | 2022-10-19 14:06:16 -0300 |
---|---|---|
committer | Kelvin M. Klann <kmk3.code@protonmail.com> | 2022-11-21 17:15:29 -0300 |
commit | 60a95f00bb1a3185fce4bacb5367201685767444 (patch) | |
tree | e4f7d7ab251be37f354d8626c1c6a331c0b857b8 /src/common.mk | |
parent | makefiles: rename H_FILE_LIST and C_FILE_LIST (diff) | |
download | firejail-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.mk | 6 |
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 | ||
5 | HDRS = $(sort $(wildcard *.h)) | 5 | HDRS := $(sort $(wildcard *.h)) |
6 | SRCS = $(sort $(wildcard *.c)) | 6 | SRCS := $(sort $(wildcard *.c)) |
7 | OBJS = $(SRCS:.c=.o) | 7 | OBJS := $(SRCS:.c=.o) |
8 | 8 | ||
9 | CFLAGS += -ggdb $(HAVE_FATAL_WARNINGS) -O2 -DVERSION='"$(VERSION)"' $(HAVE_GCOV) | 9 | CFLAGS += -ggdb $(HAVE_FATAL_WARNINGS) -O2 -DVERSION='"$(VERSION)"' $(HAVE_GCOV) |
10 | CFLAGS += -DPREFIX='"$(prefix)"' -DSYSCONFDIR='"$(sysconfdir)/firejail"' -DLIBDIR='"$(libdir)"' -DBINDIR='"$(bindir)"' -DVARDIR='"/var/lib/firejail"' | 10 | CFLAGS += -DPREFIX='"$(prefix)"' -DSYSCONFDIR='"$(sysconfdir)/firejail"' -DLIBDIR='"$(libdir)"' -DBINDIR='"$(bindir)"' -DVARDIR='"/var/lib/firejail"' |