Skip to content

Conversation

Albkat
Copy link
Member

@Albkat Albkat commented Nov 12, 2023

Revision of the CMake build:

  • Clean up CMakeLists.txt configuration files
  • Make build again version-independent: new versions of cmake (=>3.27) have problems managing dependencies of object targets (see issue)

Signed-off-by: albert <92109627+Albkat@users.noreply.github.com>

some update

Signed-off-by: albert <92109627+Albkat@users.noreply.github.com>

refactor CMake build

Signed-off-by: albert <92109627+Albkat@users.noreply.github.com>
@Albkat Albkat added the refactor Enhancing and optimizing source code for improved readability, maintainability, and performance label Nov 12, 2023
@Albkat Albkat requested review from awvwgk and MtoLStoN November 12, 2023 19:56
@Albkat Albkat merged commit fc056d1 into grimme-lab:main Nov 22, 2023
@Albkat Albkat deleted the cmake branch November 22, 2023 16:36

# Setup the XTB Project
# Buggy CMake versions
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It was fixed since 3.27.9 and in 3.28 it should works now.
See https://gitlab.kitware.com/cmake/cmake/-/issues/25425

gorges97 pushed a commit to gorges97/xtb that referenced this pull request Mar 27, 2024
* refactor CMake build

Signed-off-by: albert <92109627+Albkat@users.noreply.github.com>

some update

Signed-off-by: albert <92109627+Albkat@users.noreply.github.com>

refactor CMake build

Signed-off-by: albert <92109627+Albkat@users.noreply.github.com>

* Update CMakeLists.txt

* allow cmake v 3.17 or newer

---------

Signed-off-by: albert <92109627+Albkat@users.noreply.github.com>
Signed-off-by: Johannes Gorges <58849467+gorges97@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor Enhancing and optimizing source code for improved readability, maintainability, and performance
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants