From 3ecaea861f640d6ff9b3234f496802e5b5421a31 Mon Sep 17 00:00:00 2001 From: Matheus Afonso Martins Moreira Date: Wed, 18 Sep 2024 00:23:27 -0300 Subject: [PATCH] make: add missing directories dependency The generated NR.c and NR.list files as well as the intrinsic Linux lone lisp module did not depend on the directories phony target which sometimes causes issues in parallel builds because make can attempt to create files inside the build tree before it exists. This has been happening in the lone continuous integration builds, causing them to fail for weird reasons. --- GNUmakefile | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/GNUmakefile b/GNUmakefile index 597f641..4df0932 100644 --- a/GNUmakefile +++ b/GNUmakefile @@ -127,12 +127,12 @@ $(directories.build.tools)/%: $(directories.build.objects.tools)/%.o $(targets.o $(directories.build.tests)/%: $(directories.build.objects.tests)/%.o $(targets.objects.lone) | directories $(strip $(CC) $(flags.executable) $(CFLAGS.with_overrides) $(LDFLAGS) -o $@ $^) -$(call source_to_object,source/lone/lisp/modules/intrinsic/linux.c): $(targets.NR.c) +$(call source_to_object,source/lone/lisp/modules/intrinsic/linux.c): $(targets.NR.c) | directories -$(targets.NR.c): $(targets.NR.list) scripts/NR.generate +$(targets.NR.c): $(targets.NR.list) scripts/NR.generate | directories scripts/NR.generate < $< > $@ -$(targets.NR.list): scripts/NR.filter +$(targets.NR.list): scripts/NR.filter | directories $(CC) -E -dM -include linux/unistd.h - < /dev/null | scripts/NR.filter > $@ targets.phony += lone