Skip to content

Commit

Permalink
Move SYMREADER calculation to base CMakeLists.txt
Browse files Browse the repository at this point in the history
It is needed in several places before DyninstLibrary is included.
  • Loading branch information
hainest committed Dec 22, 2022
1 parent 238c6e1 commit ecd39e3
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 6 deletions.
6 changes: 6 additions & 0 deletions CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,12 @@ list(INSERT CMAKE_MODULE_PATH 0 "${PROJECT_SOURCE_DIR}/cmake"
# Handle user options first
include(DyninstOptions)

if(LIGHTWEIGHT_SYMTAB)
set(SYMREADER symLite)
else()
set(SYMREADER symtabAPI)
endif()

# Set up Dyninst internals
include(DyninstPlatform)
include(DyninstCapArchDef)
Expand Down
6 changes: 0 additions & 6 deletions cmake/DyninstLibrary.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -13,12 +13,6 @@ set(CMAKE_C_VISIBILITY_PRESET hidden)
set(CMAKE_CXX_VISIBILITY_PRESET hidden)
set(CMAKE_VISIBILITY_INLINES_HIDDEN ON)

if(LIGHTWEIGHT_SYMTAB)
set(SYMREADER symLite)
else()
set(SYMREADER symtabAPI)
endif()

# Link libraries privately when possible
function(target_link_private_libraries target)
if(${CMAKE_VERSION} VERSION_LESS "2.8.7")
Expand Down

0 comments on commit ecd39e3

Please sign in to comment.