Moved configure_file back to base CMakeListst

The ordering matters until common gets an explicit list of header files (currently uses a glob).
dyninst · Dec 22, 2022 · 9cd0771 · 9cd0771
1 parent 159b2b6
commit 9cd0771
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 2 deletions.
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -63,6 +63,7 @@ else()
 	add_library(OpenMP::OpenMP_CXX INTERFACE IMPORTED)
 endif()
 
+configure_file(cmake/version.h.in common/h/dyninstversion.h)
 include_directories(${PROJECT_BINARY_DIR})
 include_directories(${PROJECT_BINARY_DIR}/common/h)
 set(HEADER_DIRS

diff --git a/cmake/DyninstInstall.cmake b/cmake/DyninstInstall.cmake
@@ -4,8 +4,6 @@ include(CMakePackageConfigHelpers)
 install(DIRECTORY ${DYNINST_ROOT}/cmake/Modules DESTINATION ${INSTALL_CMAKE_DIR})
 install(DIRECTORY ${DYNINST_ROOT}/cmake/tpls DESTINATION ${INSTALL_CMAKE_DIR})
 
-configure_file(cmake/version.h.in common/h/dyninstversion.h)
-
 configure_package_config_file(
     ${PROJECT_SOURCE_DIR}/cmake/${PROJECT_NAME}Config.cmake.in
     ${PROJECT_BINARY_DIR}/${PROJECT_NAME}Config.cmake