Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/83198?usp=email )
Change subject: Makefiles: Add site_local-target to run early in the build ......................................................................
Makefiles: Add site_local-target to run early in the build
This double-colon target doesn't do anything unless it's implemented by another makefile. It's intended to be used only by the site-local makefile to allow it to run any necessary steps before the actual coreboot build begins.
Signed-off-by: Martin Roth gaumless@gmail.com Change-Id: I01f98c9cf8375bca21ab87f9becf66a25402c758 Reviewed-on: https://review.coreboot.org/c/coreboot/+/83198 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Matt DeVillier matt.devillier@amd.corp-partner.google.com Reviewed-by: Felix Held felix-coreboot@felixheld.de --- M Makefile M Makefile.mk 2 files changed, 8 insertions(+), 2 deletions(-)
Approvals: build bot (Jenkins): Verified Felix Held: Looks good to me, approved Matt DeVillier: Looks good to me, but someone else must approve
diff --git a/Makefile b/Makefile index bc6fabc..cc73900 100644 --- a/Makefile +++ b/Makefile @@ -222,8 +222,7 @@
# The primary target needs to be here before we include the # other files - -real-all: real-target +real-all: site-local-target real-target
# must come rather early .SECONDARY: diff --git a/Makefile.mk b/Makefile.mk index a16089d..bae34e0 100644 --- a/Makefile.mk +++ b/Makefile.mk @@ -90,6 +90,13 @@ .PHONY: show_notices show_notices:: | show_coreboot
+# This rule allows the site-local makefile to run before starting the actual +# coreboot build. It should not be used in the regular coreboot makefiles. +# Note: This gets run after the immediate makefile code like updating the +# submodules, but before any other targets. +.PHONY: site-local-target +site-local-target:: + ####################################################################### # our phony targets PHONY+= clean-abuild coreboot check-style build_complete