Skip to content

Commit

Permalink
Fix nasa#2128, use correct loop var in add_cfe_app_dependency
Browse files Browse the repository at this point in the history
This function was not using the loop variable, so ended up adding
only the first item.
  • Loading branch information
jphickey committed Jul 29, 2022
1 parent 924b2be commit dae6c22
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions cmake/arch_build.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -124,8 +124,8 @@ function(add_cfe_app_dependency MODULE_NAME DEPENDENCY_MODULE)
set(INCLUDE_LIST)
set(COMPILE_DEF_LIST)
foreach(DEP ${DEPENDENCY_MODULE} ${ARGN})
list(APPEND INCLUDE_LIST "$<TARGET_PROPERTY:${DEPENDENCY_MODULE},INTERFACE_INCLUDE_DIRECTORIES>")
list(APPEND COMPILE_DEF_LIST "$<TARGET_PROPERTY:${DEPENDENCY_MODULE},INTERFACE_COMPILE_DEFINITIONS>")
list(APPEND INCLUDE_LIST "$<TARGET_PROPERTY:${DEP},INTERFACE_INCLUDE_DIRECTORIES>")
list(APPEND COMPILE_DEF_LIST "$<TARGET_PROPERTY:${DEP},INTERFACE_COMPILE_DEFINITIONS>")
endforeach()

target_include_directories(${MODULE_NAME} PUBLIC
Expand Down Expand Up @@ -722,4 +722,3 @@ function(process_arch SYSVAR)
endforeach(TGTNAME ${TGTSYS_${SYSVAR}})

endfunction(process_arch SYSVAR)

0 comments on commit dae6c22

Please sign in to comment.