diff options
| author | Nico Huber <nico.h@gmx.de> | 2020-05-29 00:50:47 +0200 |
|---|---|---|
| committer | Patrick Georgi <pgeorgi@google.com> | 2020-06-10 21:26:23 +0000 |
| commit | 91844b1f358e3ed3f043ee7c6f53ba8f4567a088 (patch) | |
| tree | 547c1834b6034749f2e66a31321eb65c1200c7df /util/abuild | |
| parent | ad9f8cf02d1996bacaad584543a68122df822031 (diff) | |
| download | coreboot-91844b1f358e3ed3f043ee7c6f53ba8f4567a088.tar.xz | |
Revert "util/abuild: Have abuild generate the .xcompile if it doesn't exist"
This reverts commit 9ff4029db9b1053b44e4fc487243017a099a49c5.
Pulling the toplevel Makefile into a tiny one has all sorts of side
effects. For instance, the toplevel (random) .config is also included
so the results depend on the board that is selected there. What finally
broke it is a line that is unconditionally printed for AMD Picasso
boards resulting in lots of lines like this:
skipping LENOVO_W520 because we're missing compilers for \
(Adding PSP c7ce61492157d3237f679c4a40a08b79 \
.../coreboot/3rdparty/amd_blobs/picasso/PSP/PspBootLoader_prod_RV.sbin)
While both issues, the random .config and amd/picasso, could be worked
around easily, it seems hard to predict what other pitfalls are lurking
in the Makefile inclusion. Also, the problem solved by its inclusion
can be fixed by a much simpler `make .xcompile`.
Change-Id: I2ff70f561d717eb30e5f3c06c83e83468e174ec5
Signed-off-by: Nico Huber <nico.h@gmx.de>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/41846
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-by: Patrick Georgi <pgeorgi@google.com>
Diffstat (limited to 'util/abuild')
| -rwxr-xr-x | util/abuild/abuild | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/util/abuild/abuild b/util/abuild/abuild index 934bc5d4ac..022567da59 100755 --- a/util/abuild/abuild +++ b/util/abuild/abuild @@ -466,14 +466,13 @@ function build_config required_arches=$(grep -E "^CONFIG_ARCH_(BOOTBLOCK|R.MSTAGE|VERSTAGE)" "$TARGET/${BUILD_NAME}/config.build" | \ sed "s,^CONFIG_ARCH_[^_]*_\([^=]*\)=.*$,\1," |sort -u |tr 'A-Z\n\r' 'a-z ') - missing_arches="$($MAKE --no-print-directory -f Makefile -f - \ - REQUIRED_ARCHES="$required_arches" obj="${build_dir}" <<'EOF' + missing_arches="$($MAKE --no-print-directory -f - \ + REQUIRED_ARCHES="$required_arches" <<'EOF' include .xcompile .PHONY: missing_arches missing_arches: $(if $(XCOMPILE_COMPLETE),,$(error .xcompile is invalid.)) @echo $(foreach arch,$(REQUIRED_ARCHES),$(if $(filter $(arch),$(SUBARCH_SUPPORTED)),,$(arch))) -.DEFAULT_GOAL := missing_arches EOF )" # shellcheck disable=SC2181 |