Skip to content

Commit

Permalink
Merge pull request #4142 from vicentebolea/fix-pugixml-dep
Browse files Browse the repository at this point in the history
pugixml,cmake: fix linkage when pugixml is external
  • Loading branch information
vicentebolea authored Apr 21, 2024
2 parents 0ee43a4 + f1322a3 commit 6102cab
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 15 deletions.
12 changes: 4 additions & 8 deletions source/adios2/toolkit/remote/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -6,15 +6,11 @@
if (NOT ADIOS2_USE_PIP)
add_executable(adios2_remote_server ./remote_server.cpp remote_common.cpp)

target_link_libraries(adios2_remote_server PUBLIC EVPath::EVPath adios2_core adios2sys
PRIVATE $<$<PLATFORM_ID:Windows>:shlwapi>)
target_link_libraries(adios2_remote_server
PUBLIC EVPath::EVPath adios2_core adios2sys
PRIVATE adios2::thirdparty::pugixml $<$<PLATFORM_ID:Windows>:shlwapi>)

get_property(pugixml_headers_path
TARGET pugixml
PROPERTY INTERFACE_INCLUDE_DIRECTORIES
)

target_include_directories(adios2_remote_server PRIVATE ${PROJECT_BINARY_DIR} ${pugixml_headers_path})
target_include_directories(adios2_remote_server PRIVATE ${PROJECT_BINARY_DIR})

set_property(TARGET adios2_remote_server PROPERTY OUTPUT_NAME adios2_remote_server${ADIOS2_EXECUTABLE_SUFFIX})
install(TARGETS adios2_remote_server EXPORT adios2
Expand Down
8 changes: 1 addition & 7 deletions source/utils/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -13,17 +13,11 @@ configure_file(
add_executable(bpls ./bpls/bpls.cpp)
target_link_libraries(bpls
PUBLIC adios2_core adios2sys
PRIVATE $<$<PLATFORM_ID:Windows>:shlwapi>)

get_property(pugixml_headers_path
TARGET pugixml
PROPERTY INTERFACE_INCLUDE_DIRECTORIES
)
PRIVATE adios2::thirdparty::pugixml $<$<PLATFORM_ID:Windows>:shlwapi>)

target_include_directories(bpls PRIVATE
${PROJECT_BINARY_DIR}
${PROJECT_SOURCE_DIR}/bindings/C
${pugixml_headers_path}
)

set_property(TARGET bpls PROPERTY OUTPUT_NAME bpls${ADIOS2_EXECUTABLE_SUFFIX})
Expand Down

0 comments on commit 6102cab

Please sign in to comment.