-
Notifications
You must be signed in to change notification settings - Fork 173
Doc updates for CMake modernization #1341
Copy link
Copy link
Closed
Labels
Description
- Users who don't want to completely update to target-based CMake will need to add
if(TARGET Dyninst::Dyninst)
foreach(t common symtabapi ...)
# we could use `add_library(${t} ALIAS Dyninst::${t}) if Dyninst::${t} is a GLOBAL imported target
# See https://cmake.org/cmake/help/latest/command/add_library.html#alias-libraries
add_library(${t} INTERFACE IMPORTED)
target_link_libraries(${t} INTERFACE Dyninst::${t})
endforeach()
endif()
Changes to DyninstConfig.cmake
DYNINST_LIBRARIESis gone. It only listed "dyninstAPI".
Reactions are currently unavailable