Attention is currently required from: Bora Guvendik, Nico Huber, Wonkyu Kim, Jérémy Compostella, Nicholas Chin.
Selma Bensaid has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/67118 )
Change subject: libpayload/Makefile.inc: Initialize vboot depending on UPDATED_SUBMODULES ......................................................................
Patch Set 3:
(1 comment)
File payloads/libpayload/Makefile.inc:
https://review.coreboot.org/c/coreboot/+/67118/comment/a3542765_2b26b7fd PS2, Line 91: ifneq ($(UPDATED_SUBMODULES),1)
Apparently one purpose of UPDATED_SUBMODULES in coreboot's top level Makefile. […]
Hi Nico, are you building manually libpayload? Here what I see when building libpayload manually (I added few logs to the Makefile.inc) sbensaid@sbensaid-Z97X-UD7-TH:/tmp/coreboot_build/payloads/libpayload$ make @@@@@@@@@@@@@@@@@@@@@@@@@ UPDATED_SUBMODULES @@@@@@@@@@@@@@@@@@@@@@@@@ UPDATED_SUBMODULES is not defined @@@@@@@@@@@@@@@@@@@@@@@@@ UPDATED_SUBMODULES 0 @@@@@@@@@@@@@@@@@@@@@@@@@ UPDATED_SUBMODULES synching
bensaid@sbensaid-Z97X-UD7-TH:/tmp/coreboot_build/payloads/libpayload$ UPDATED_SUBMODULES=1 make @@@@@@@@@@@@@@@@@@@@@@@@@ UPDATED_SUBMODULES 1 @@@@@@@@@@@@@@@@@@@@@@@@@ UPDATED_SUBMODULES 1
sbensaid@sbensaid-Z97X-UD7-TH:/tmp/coreboot_build/payloads/libpayload$ UPDATED_SUBMODULES=0 make @@@@@@@@@@@@@@@@@@@@@@@@@ UPDATED_SUBMODULES 0 @@@@@@@@@@@@@@@@@@@@@@@@@ UPDATED_SUBMODULES 0 @@@@@@@@@@@@@@@@@@@@@@@@@ UPDATED_SUBMODULES synching
I did not export UPDATED_SUBMODULES