From 56c3be2604e790275c47a077c4b84a4fb3d75746 Mon Sep 17 00:00:00 2001 From: Ferdinand Bachmann Date: Tue, 13 Jan 2026 15:19:43 +0100 Subject: [PATCH] fix(cmake): Set CACHE variables correctly The syntax for setting cache variables is actually set( CACHE ) and not set( CACHE ). The previous code silently set the variables to the empty string. --- components/mbedtls/CMakeLists.txt | 6 +++--- components/mbedtls/esp_tee/esp_tee_mbedtls.cmake | 6 +++--- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/components/mbedtls/CMakeLists.txt b/components/mbedtls/CMakeLists.txt index 9db36e5127..69ed9564ef 100644 --- a/components/mbedtls/CMakeLists.txt +++ b/components/mbedtls/CMakeLists.txt @@ -124,11 +124,11 @@ if(CONFIG_MBEDTLS_CERTIFICATE_BUNDLE) endif() # Only build mbedtls libraries -set(ENABLE_TESTING CACHE BOOL OFF) -set(ENABLE_PROGRAMS CACHE BOOL OFF) +set(ENABLE_TESTING OFF CACHE BOOL "mbedtls: enable testing") +set(ENABLE_PROGRAMS OFF CACHE BOOL "mbedtls: enable programs") # Use pre-generated source files in mbedtls repository -set(GEN_FILES CACHE BOOL OFF) +set(GEN_FILES OFF CACHE BOOL "mbedtls: use pre-generated source files") # Make sure mbedtls finds the same Python interpreter as IDF uses idf_build_get_property(python PYTHON) diff --git a/components/mbedtls/esp_tee/esp_tee_mbedtls.cmake b/components/mbedtls/esp_tee/esp_tee_mbedtls.cmake index 70596b4182..2d6b1f5427 100644 --- a/components/mbedtls/esp_tee/esp_tee_mbedtls.cmake +++ b/components/mbedtls/esp_tee/esp_tee_mbedtls.cmake @@ -19,11 +19,11 @@ idf_component_register(SRCS "${srcs}" PRIV_REQUIRES "${priv_requires}") # Only build mbedtls libraries -set(ENABLE_TESTING CACHE BOOL OFF) -set(ENABLE_PROGRAMS CACHE BOOL OFF) +set(ENABLE_TESTING OFF CACHE BOOL "mbedtls: enable testing") +set(ENABLE_PROGRAMS OFF CACHE BOOL "mbedtls: enable programs") # Use pre-generated source files in mbedtls repository -set(GEN_FILES CACHE BOOL OFF) +set(GEN_FILES OFF CACHE BOOL "mbedtls: use pre-generated source files") # Needed to for include_next includes to work from within mbedtls include_directories("${COMPONENT_DIR}/port/include")