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

Reply via email to