michaelh/zephyr
1
0
Fork 0
Code Releases Activity

cmake: Fix variable casing mixup

Browse source 
A variable called llext_cflags was mixed up with another called
LLEXT_CFLAGS, and that caused some host directories to end up leaking
unnecessarily into the EDK.

Fix by using only the lower case version.

Signed-off-by: Ederson de Souza <ederson.desouza@intel.com>
This commit is contained in:
Ederson de Souza 2024-05-23 16:38:15 -07:00 • committed by Anas Nashif
commit bc9a1be976
1 changed files with 2 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

4
cmake/llext-edk.cmake
View file

@ -94,7 +94,7 @@ string(REGEX REPLACE "[^a-zA-Z0-9]" "_" llext_edk_name_sane ${llext_edk_name})
string(TOUPPER ${llext_edk_name_sane} llext_edk_name_sane) string(TOUPPER ${llext_edk_name_sane} llext_edk_name_sane)
set(install_dir_var "${llext_edk_name_sane}_INSTALL_DIR") set(install_dir_var "${llext_edk_name_sane}_INSTALL_DIR")
separate_arguments(LLEXT_CFLAGS NATIVE_COMMAND ${llext_cflags}) separate_arguments(llext_cflags NATIVE_COMMAND ${llext_cflags})
set(make_relative FALSE) set(make_relative FALSE)
foreach(flag ${llext_cflags}) foreach(flag ${llext_cflags})
@ -117,7 +117,7 @@ foreach(flag ${llext_cflags})
list(APPEND new_cflags ${flag}) list(APPEND new_cflags ${flag})
endif() endif()
endforeach() endforeach()
set(LLEXT_CFLAGS ${new_cflags}) set(llext_cflags ${new_cflags})
cmake_path(CONVERT "${INTERFACE_INCLUDE_DIRECTORIES}" TO_CMAKE_PATH_LIST include_dirs) cmake_path(CONVERT "${INTERFACE_INCLUDE_DIRECTORIES}" TO_CMAKE_PATH_LIST include_dirs)