[coreboot-gerrit] Change in coreboot[master]: util/abuild: Run `make .xcompile` only once

[Felix Held (Code Review)]

Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/42640 )

Change subject: util/abuild: Run `make .xcompile` only once
......................................................................

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 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/42640
Tested-by: build bot (Jenkins) 
Reviewed-by: Tim Wawrzynczak 
Reviewed-by: Reka Norman 
Reviewed-by: Patrick Georgi 
---
M util/abuild/abuild
1 file changed, 4 insertions(+), 4 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Patrick Georgi: Looks good to me, approved
  Tim Wawrzynczak: Looks good to me, but someone else must approve
  Reka Norman: Looks good to me, but someone else must approve

diff --git a/util/abuild/abuild b/util/abuild/abuild
index 5fe298e..dc38efe 100755
--- a/util/abuild/abuild
+++ b/util/abuild/abuild
@@ -786,10 +786,6 @@
    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 @@
 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