Skip to content

Reworking the cmake infrastructure

Nick R. Papior requested to merge npapior/siesta:cmake-upgrades into master

This should alleviate some of the decisions that users needs to make. For instance instead of forcing users to take a stance on every single library, we instead enable the library IFF it is accessible and usable. There might be certain libraries that are not defaulted to be compiled in, in which case the default should not be TRUE. However, in those cases we should clearly spell out that the users needs to take a decision.

Beautification of the output to a great extent. Many places I have added an indentation to make grouping a little softer on the reader. This could be expanded further.

The search_.cmake files have been moved to FindCustom.cmake to make use of find_package. This will allow one to test whether a package exists, and then make a decision based on whether it does or not.

Added the BLAS library in case LAPACK does not add it. BLAS is not required if LAPACK has the functionality required. This is mainly to allow sub-packages to rely on BLAS-only.

Once the build-setup has completed, it will print-out information about the build-system for the end-user. This simplifies the experience by clearly specifying which variables are used, which are not and whether the feature has been enabled.

Changes of names:

  • WITH_LUA is changed to WITH_FLOOK
  • internal target: Scalapack::Scalapack is changed to SCALAPACK::SCALAPACK for consistency.

Signed-off-by: Nick Papior nickpapior@gmail.com

Edited by Nick R. Papior

Merge request reports