Attention is currently required from: Martin L Roth.
Nikolai Vyssotski has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/68799 )
Change subject: Makefile.inc: site-local path must go first ......................................................................
Makefile.inc: site-local path must go first
"site-local" Makfile(s) may need to override some of the macros/paths used elsewhere in src/* makefiles. If we include it last src/* Makefile.inc will have already been processed. MAINBOARD_BLOBS_DIR is an example where the path needs to be overwritten in site-local requiring it to be included first before src/mainboard/* Makefile.inc is processed.
Change-Id: I8ea865cd73aba5092a628b0422e5c4121b32fb4d --- M Makefile.inc 1 file changed, 19 insertions(+), 2 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/99/68799/1
diff --git a/Makefile.inc b/Makefile.inc index 0ed205f..c624ffe 100644 --- a/Makefile.inc +++ b/Makefile.inc @@ -79,7 +79,9 @@
####################################################################### # root source directories of coreboot -subdirs-y := src/lib src/commonlib/ src/console src/device src/acpi src/superio/common +# site-local Makefile.inc must go first to override default locations (for binaries etc.) +subdirs-y := site-local +subdirs-y += src/lib src/commonlib/ src/console src/device src/acpi src/superio/common subdirs-y += src/ec/acpi $(wildcard src/ec/*/*) $(wildcard src/southbridge/*/*) subdirs-y += $(wildcard src/soc/*) $(wildcard src/soc/*/common) $(filter-out $(wildcard src/soc/*/common),$(wildcard src/soc/*/*)) subdirs-y += $(wildcard src/northbridge/*/*) @@ -95,7 +97,6 @@ subdirs-y += payloads payloads/external subdirs-$(CONFIG_SBOM) += src/sbom
-subdirs-y += site-local subdirs-y += util/checklist util/testing
#######################################################################
-
Nikolai Vyssotski (Code Review)