Branch: refs/heads/cmake_modernization Home: https://github.com/dyninst/dyninst Commit: e334905021ee5ae1c9d9e3af6657150cbde968a4 https://github.com/dyninst/dyninst/commit/e334905021ee5ae1c9d9e3af6657150cbde968a4 Author: Tim Haines <thaines.as...@gmail.com> Date: 2022-12-02 (Fri, 02 Dec 2022)
Changed paths: M CMakeLists.txt M cmake/DyninstConfig.cmake.in R cmake/Modules/FindTBB.cmake M cmake/tpls/DyninstTBB.cmake R cmake/tpls/DyninstTBBInstall.cmake M common/CMakeLists.txt M dyninstAPI/CMakeLists.txt M dyninstAPI_RT/CMakeLists.txt M elf/CMakeLists.txt M instructionAPI/CMakeLists.txt M symtabAPI/CMakeLists.txt Log Message: ----------- Require CMake package for TBB (#1322) * Make libdl/dbghelp private linkage * Remove TBB flags from toolkits that don't use TBB * Remove FindTBB.cmake All supported TBB versions ship as CMake packages, so this is no longer needed. * Remove from-source build option * Create an imported target for TBB used by Dyninst This is needed to force the include directories to be considered 'system' directories so that compiler warnings from TBB sources are ignored * Rename cmake/ThreadingBuildingBlocks.cmake -> cmake/tpls/DyninstTBB.cmake This is needed to keep the namespace clean for DyninstConfig.cmake * Export TBB as part of the Dyninst CMake package This is required by the CMake guidelines: https://cmake.org/cmake/help/latest/guide/importing-exporting/index.html In particular, "All required dependencies of a package must also be found in the package configuration file" * Format DyninstTBB.cmake _______________________________________________ Dyninst-api mailing list Dyninst-api@cs.wisc.edu https://lists.cs.wisc.edu/mailman/listinfo/dyninst-api