Skip to content

Commit

Permalink
Merge pull request #2129 from jphickey/fix-2128-add-dependency
Browse files Browse the repository at this point in the history
Fix #2128, use correct loop var in add_cfe_app_dependency
  • Loading branch information
dzbaker committed Aug 4, 2022
2 parents 924b2be + dae6c22 commit c2db636
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 c2db636

Please sign in to comment.