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

Updated Cascade to build with spdlog 1.12 #77

Merged
merged 2 commits into from
Nov 15, 2024
Merged

Updated Cascade to build with spdlog 1.12 #77

merged 2 commits into from
Nov 15, 2024

Conversation

etremel
Copy link
Contributor

@etremel etremel commented Oct 18, 2024

The next version of Derecho will depend on spdlog 1.12 instead of spdlog 1.9.2, and this library update introduced breaking changes. A few new lines of code are needed to allow Cascade to compile against the Derecho version that uses spdlog 1.12, and Cascade's own CMakeLists should specify that it expects the newer spdlog version (since it will no longer compile against spdlog 1.9.2 with these changes). Once there is a Derecho version that uses the new spdlog, we can merge these changes and create a Cascade version that matches the dependency.

The next version of Derecho will depend on spdlog 1.12 instead of
spdlog 1.9.2, and this library update introduced breaking changes. A few
new lines of code are needed to allow Cascade to compile against the
Derecho version that uses spdlog 1.12, and Cascade's own CMakeLists
should specify that it expects the newer spdlog version (since it will
no longer compile against spdlog 1.9.2 with these changes).
Now that the spdlog updates are in version 2.4.1 of Derecho, the README
should say that the current version of Cascade requires Derecho 2.4.1.
Also, when writing the package config file, we should use the "full"
version (cascade_build_VERSION) for the VERSION property to make sure
the patch level is included, as documented here:
https://cmake.org/cmake/help/latest/module/CMakePackageConfigHelpers.html#generating-a-package-version-file
@etremel
Copy link
Contributor Author

etremel commented Nov 5, 2024

Note: Just like in the related Derecho PR (283), the build check "failed" on this because the ubuntu-latest image on GitHub's build servers is still 22.04 and cannot satisfy the dependency on spdlog 1.12. This actually builds just fine on Ubuntu 24.04, or on Fractus servers with spdlog 1.12 manually installed. (There is a GitHub issue noting that ubuntu-latest should be 24.04 by now but it has been put on hold).

@etremel etremel requested a review from tgarr November 14, 2024 15:13
@tgarr tgarr merged commit 22cbe3f into master Nov 15, 2024
1 check failed
@etremel etremel deleted the spdlog_update branch November 19, 2024 15:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants