Raul Rangel has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/52578 )
Change subject: Makefile,tests: Move cmocka checkout into top level Makefile ......................................................................
Makefile,tests: Move cmocka checkout into top level Makefile
cmocka is currently ignoring the UPDATED_SUBMODULES flag. Move the cmocka checkout with the other submodule checkouts.
BUG=none TEST=Make sure cmocka is not checked out if UPDATED_SUBMODULES=1
Signed-off-by: Raul E Rangel rrangel@chromium.org Change-Id: I2a1db809368a77d2c0f9c9a796d62555ec476dc7 --- M Makefile.inc M tests/Makefile.inc 2 files changed, 2 insertions(+), 3 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/78/52578/1
diff --git a/Makefile.inc b/Makefile.inc index a372efa..ce70a30 100644 --- a/Makefile.inc +++ b/Makefile.inc @@ -193,6 +193,8 @@ ifneq ($(UPDATED_SUBMODULES),1) # try to fetch non-optional submodules if the source is under git forgetthis:=$(if $(GIT),$(shell git submodule update --init)) +# Checkout Cmocka repository +forgetthis:=$(if $(GIT),$(shell git submodule update --init --checkout 3rdparty/cmocka)) ifeq ($(CONFIG_USE_BLOBS),y) # These items are necessary because each has update=none in .gitmodules. They are ignored # until expressly requested and enabled with --checkout diff --git a/tests/Makefile.inc b/tests/Makefile.inc index 48bd18b..fd3cd9f 100644 --- a/tests/Makefile.inc +++ b/tests/Makefile.inc @@ -41,9 +41,6 @@
TEST_CFLAGS += -D__TEST__
-# Checkout Cmocka repository -forgetthis:=$(shell git submodule update --init --checkout 3rdparty/cmocka) - TEST_CFLAGS += -I$(cmockasrc)/include
# Link against Cmocka