Skip to content

Doc updates for CMake modernization #1341

@hainest

Description

@hainest
  1. 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

  1. DYNINST_LIBRARIES is gone. It only listed "dyninstAPI".

Metadata

Metadata

Assignees

Labels

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions