[DynInst_API:] [dyninst/dyninst] e33490: Require CMake package for TBB (#1322)


Date: Fri, 02 Dec 2022 09:58:13 -0800
From: Tim Haines <noreply@xxxxxxxxxx>
Subject: [DynInst_API:] [dyninst/dyninst] e33490: Require CMake package for TBB (#1322)
  Branch: refs/heads/cmake_modernization
  Home:   https://github.com/dyninst/dyninst
  Commit: e334905021ee5ae1c9d9e3af6657150cbde968a4
      https://github.com/dyninst/dyninst/commit/e334905021ee5ae1c9d9e3af6657150cbde968a4
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  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


[← Prev in Thread] Current Thread [Next in Thread→]
  • [DynInst_API:] [dyninst/dyninst] e33490: Require CMake package for TBB (#1322), Tim Haines <=