Skip to content

Cmake improvements

KTGH requested to merge github/fork/sum01/cmake_bonanza into master

What this PR does:

  • Fixes the minimum Cmake version, because cxx_std_17 was already in use.
  • Optional use of system libraries instead of (mostly) git submodule libs.
  • Improves flexibility of installation dirs by using GNUInstallDirs
  • Speeds up recompilation by not rebuilding unchanged files.
  • Fine-tunes the compiler options/definitions
  • Allows for find_package(Acid) to work on systems that already have Acid installed. This will import an Acid::Acid target for linking.
  • Add CTest support for automatically running the tests, and add an option ACID_INSTALL_EXAMPLES to not install said tests.

It does a little more, but I tried to write meaningful commit messages for that reason :)

I would love some notes on if I missed anything, or accidentally mangled previously working features. Hopefully that didn't happen, but I might have missed something.

TODO: Use CTest to automatically test the tests, and add an option to not install said tests.

Merge request reports