Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor CMake build #900

Merged
merged 3 commits into from
Nov 22, 2023
Merged

refactor CMake build #900

merged 3 commits into from
Nov 22, 2023

Conversation

Albkat
Copy link
Contributor

@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 merged commit fc056d1 into grimme-lab:main Nov 22, 2023
10 of 11 checks passed
@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.

None yet

3 participants