[DynInst_API:] [dyninst/dyninst] c981db: Boost CMake modernization (#1330)


Date: Wed, 07 Dec 2022 09:58:21 -0800
From: Tim Haines <noreply@xxxxxxxxxx>
Subject: [DynInst_API:] [dyninst/dyninst] c981db: Boost CMake modernization (#1330)
  Branch: refs/heads/cmake_modernization
  Home:   https://github.com/dyninst/dyninst
  Commit: c981db140d919042013160b9db0a6bc9b8b80bd9
      https://github.com/dyninst/dyninst/commit/c981db140d919042013160b9db0a6bc9b8b80bd9
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2022-12-07 (Wed, 07 Dec 2022)

  Changed paths:
    M cmake/DyninstConfig.cmake.in
    M cmake/DyninstLibrary.cmake
    R cmake/Modules/FindBoost.cmake
    M cmake/tpls/DyninstBoost.cmake
    M common/CMakeLists.txt
    M dwarf/CMakeLists.txt
    M dyninstAPI/CMakeLists.txt
    M elf/CMakeLists.txt
    M instructionAPI/CMakeLists.txt
    M parseAPI/CMakeLists.txt
    M parseThat/CMakeLists.txt
    M patchAPI/CMakeLists.txt
    M proccontrol/CMakeLists.txt
    M stackwalk/CMakeLists.txt
    M symtabAPI/CMakeLists.txt

  Log Message:
  -----------
  Boost CMake modernization (#1330)

* Remove FindBoost.cmake

Use the one provided by CMake so we don't have to maintain this one.

* Remove from-source build

* Remove user-configurable version

* Force use of multithreaded libraries

We don't need to include Threads here. Boost will do that.

* Disable use of statically-linked runtime

* Remove Boost_DEBUG

* Always enable searching system paths

* Unify path calculations, pass to find_package as hints

* Clean up comments

* Use include_guard

* Allow using Boost's CMake package

It is now provided by default since 1.70.0 which is the current minimum
acceptable version.

* Remove rest of cache variables

* Remove MSVC-specific template define

We can add this back, if we find there are newer VCs still affected.

* Make Dyninst::Boost imported interface target

This is needed to make the include directories be "system" directories
so that warnings in their headers do not propagate into Dyninst.

* Do not add Boost as a dependency for all libs

* Make find_package QUIET

* Use Boost_* variables instead of calculating includes, libs, etc.

* Make a header-only wrapper target

* Add to Dyninst package

* Update CMakeLists

* Whitespace formatting

* Set Boost_NO_WARN_NEW_VERSIONS

* Bump minimum version to 1.71.0


[← Prev in Thread] Current Thread [Next in Thread→]
  • [DynInst_API:] [dyninst/dyninst] c981db: Boost CMake modernization (#1330), Tim Haines <=