summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPatrick Georgi <pgeorgi@google.com>2020-11-11 14:50:01 +0100
committerPatrick Georgi <pgeorgi@google.com>2020-11-11 19:45:47 +0000
commited820a0e21510c748363ac2ed6bce1242277fbc7 (patch)
treed475bf0bafdc0d9d6ce4c22f54e8bbc709dd8bcb
parent2a5fe1d64b012553d91355fe82b3b6b49f8812dc (diff)
util/futility: Don't refresh the binary all the time
Due to the phony dependency to check for openssl, vboot-futility was always rebuilt, and because it was newer than coreboot-futility, it was always copied over. Do that in parallel often enough and you run into race conditions, as we did on our builders. Mark check-openssl-presence as order-only dependency so that it's executed (and can bail out) but doesn't force regeneration of vboot-futility. Change-Id: Ib7fb798096d423d6b6cba5d199e12fe5917c3b41 Signed-off-by: Patrick Georgi <pgeorgi@google.com> Reviewed-on: https://review.coreboot.org/c/coreboot/+/47453 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Angel Pons <th3fanbus@gmail.com> Reviewed-by: Martin Roth <martinroth@google.com> Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org>
-rw-r--r--util/futility/Makefile.inc2
1 files changed, 1 insertions, 1 deletions
diff --git a/util/futility/Makefile.inc b/util/futility/Makefile.inc
index ed185ce0d5..9890339d46 100644
--- a/util/futility/Makefile.inc
+++ b/util/futility/Makefile.inc
@@ -4,7 +4,7 @@ VBOOT_FUTILITY = $(VBOOT_HOST_BUILD)/futility/futility
HOSTPKGCONFIG ?= pkg-config
-$(VBOOT_FUTILITY): check-openssl-presence
+$(VBOOT_FUTILITY): | check-openssl-presence
@printf " MAKE $(subst $(objutil)/,,$(@))\n"
unset CFLAGS LDFLAGS; $(MAKE) -C $(VBOOT_SOURCE) \
BUILD=$(VBOOT_HOST_BUILD) \