summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorNico Huber <nico.h@gmx.de>2020-06-21 15:04:37 +0200
committerFelix Held <felix-coreboot@felixheld.de>2021-09-24 14:32:07 +0000
commite28eeb713dcd1ec4712ba37e8cacb04131ffdb46 (patch)
tree580a991b6f8b833114c240c913e6704cb8ff732d
parent90fcffb416c6b104f7e8c32b8013f34ae7a55cc0 (diff)
util/abuild: Run `make .xcompile` only once
If abuild called itself recursively, the file already exists and we can spare us one evaluation of all the makefiles per recursive abuild run. Change-Id: Id3e2239354ec251c24c03c971987586deeb026c5 Signed-off-by: Nico Huber <nico.h@gmx.de> Reviewed-on: https://review.coreboot.org/c/coreboot/+/42640 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Tim Wawrzynczak <twawrzynczak@chromium.org> Reviewed-by: Reka Norman <rekanorman@chromium.org> Reviewed-by: Patrick Georgi <pgeorgi@google.com>
-rwxr-xr-xutil/abuild/abuild8
1 files changed, 4 insertions, 4 deletions
diff --git a/util/abuild/abuild b/util/abuild/abuild
index 5fe298ee6e..dc38efee1f 100755
--- a/util/abuild/abuild
+++ b/util/abuild/abuild
@@ -786,10 +786,6 @@ if ! mkdir -p "$TARGET"; then
exit 1
fi
-# Generate a single xcompile for all boards
-export xcompile="${TARGET}/xcompile"
-$MAKE -C"${ROOT}" obj="$TARGET/temp" objutil="$TARGET/sharedutils" UPDATED_SUBMODULES=1 "${xcompile}" || exit 1
-
customizing=$(echo "$customizing" | cut -c3-)
if [ "$customizing" = "" ]; then
customizing="default configuration"
@@ -798,7 +794,11 @@ fi
FAILED_BOARDS="$(realpath ${TARGET}/failed_boards)"
PASSED_BOARDS="$(realpath ${TARGET}/passing_boards)"
+# Generate a single xcompile for all boards
+export xcompile="${TARGET}/xcompile"
+
if [ "$recursive" = "false" ]; then
+ $MAKE -C"${ROOT}" obj="$TARGET/temp" objutil="$TARGET/sharedutils" UPDATED_SUBMODULES=1 "${xcompile}" || exit 1
rm -f "$FAILED_BOARDS" "$PASSED_BOARDS"
fi