Skip to content

Commit

Permalink
Starling third party fix (#65)
Browse files Browse the repository at this point in the history
  • Loading branch information
RReichert authored Sep 28, 2020
1 parent f29c5fa commit eadd72a
Show file tree
Hide file tree
Showing 3 changed files with 12 additions and 6 deletions.
4 changes: 2 additions & 2 deletions FindNanopb.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ include("GenericFindDependency")
option(nanopb_BUILD_GENERATOR "" OFF)
GenericFindDependency(
TARGET protobuf-nanopb
SOURCE_DIR "third_party/nanopb"
SOURCE_DIR "nanopb"
SYSTEM_INCLUDES
)
)

8 changes: 7 additions & 1 deletion FindStarling.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -7,10 +7,16 @@ option(starling_ENABLE_EXAMPLES "" OFF)
GenericFindDependency(
TARGET pvt-runner-lib
ADDITIONAL_TARGETS
math_routines
sensorfusion
pvt_driver
pvt-engine
pvt-common
pvt-engine
pvt-runner
pvt-sbp-logging
pvt-sizes
pvt-version
starling-build-config
starling-util
SOURCE_DIR starling
SYSTEM_HEADER_FILE "pvt_driver/runner/pvt_runner.h"
Expand Down
6 changes: 3 additions & 3 deletions GenericFindDependency.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -254,9 +254,7 @@ macro(create_source_search_paths)
list(APPEND x_SOURCE_SEARCH_PATHS "${PROJECT_SOURCE_DIR}/third_party/${x_TARGET}")
list(APPEND x_SOURCE_SEARCH_PATHS "${PROJECT_SOURCE_DIR}/third_party/lib${x_TARGET}")
else()
list(APPEND x_SOURCE_SEARCH_PATHS "${CMAKE_CURRENT_SOURCE_DIR}/${x_SOURCE_DIR}")
list(APPEND x_SOURCE_SEARCH_PATHS "${CMAKE_CURRENT_SOURCE_DIR}/third_party/${x_SOURCE_DIR}")
list(APPEND x_SOURCE_SEARCH_PATHS "${PROJECT_SOURCE_DIR}/${x_SOURCE_DIR}")
list(APPEND x_SOURCE_SEARCH_PATHS "${PROJECT_SOURCE_DIR}/third_party/${x_SOURCE_DIR}")
list(APPEND x_SOURCE_SEARCH_PATHS "${CMAKE_CURRENT_SOURCE_DIR}/third_party/${CMAKE_FIND_PACKAGE_NAME}/${x_SOURCE_DIR}")
list(APPEND x_SOURCE_SEARCH_PATHS "${CMAKE_CURRENT_SOURCE_DIR}/third_party/${x_TARGET}/${x_SOURCE_DIR}")
Expand Down Expand Up @@ -392,7 +390,9 @@ function(GenericFindDependency)
# If the primary target exposes additional targets, mark them as system
# targets in accordance to how the primary target was marked
foreach(additional_target IN LISTS x_ADDITIONAL_TARGETS)
mark_target_as_system_includes(${additional_target})
if(TARGET ${additional_target})
mark_target_as_system_includes(${additional_target})
endif()
endforeach()
endfunction()

0 comments on commit eadd72a

Please sign in to comment.