Skip to content

Remove deprecated support for fmt and spdlog#914

Merged
rapids-bot[bot] merged 5 commits intorapidsai:branch-25.12from
bdice:remove-fmt-spdlog
Sep 25, 2025
Merged

Remove deprecated support for fmt and spdlog#914
rapids-bot[bot] merged 5 commits intorapidsai:branch-25.12from
bdice:remove-fmt-spdlog

Conversation

@bdice
Copy link
Copy Markdown
Contributor

@bdice bdice commented Sep 24, 2025

Description

This PR removes deprecated support for fmt and spdlog. Planned in #813, deprecated in #853.

Checklist

  • I am familiar with the Contributing Guidelines.
  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.
  • The cmake-format.json is up to date with these changes.
  • I have added new files under rapids-cmake/
    • I have added include guards (include_guard(GLOBAL))
    • I have added the associated docs/ rst file and update the api.rst

@bdice bdice requested review from a team as code owners September 24, 2025 14:21
@bdice bdice requested review from AyodeAwe and removed request for a team September 24, 2025 14:21
@KyleFromNVIDIA
Copy link
Copy Markdown
Member

What is the motivation for this? Please link to the build-planning issue.

@bdice
Copy link
Copy Markdown
Contributor Author

bdice commented Sep 24, 2025

@KyleFromNVIDIA I updated the PR description (needed to look up the issue number). This has been deprecated for a while, with a removal target of 25.12.

@bdice
Copy link
Copy Markdown
Contributor Author

bdice commented Sep 24, 2025

@vyasr We'll need to fix rapids-logger so that it no longer uses these deprecated features. It looks like you've already done this in rapidsai/rapids-logger#38, but we need to update a version somewhere. I'll look a little further at this.

@bdice bdice added breaking Introduces a breaking change improvement Improves an existing functionality labels Sep 24, 2025
@bdice bdice self-assigned this Sep 24, 2025
…g rapids-logger/spdlog which by default requires <format>, which needs a newer GCC version than we can support with CUDA 12.2. Whew.
@bdice
Copy link
Copy Markdown
Contributor Author

bdice commented Sep 25, 2025

/merge

@rapids-bot rapids-bot bot merged commit 5ffba3c into rapidsai:branch-25.12 Sep 25, 2025
14 checks passed
@bdice bdice mentioned this pull request Sep 26, 2025
7 tasks
vyasr added a commit to vyasr/rapids-cmake that referenced this pull request Sep 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking Introduces a breaking change improvement Improves an existing functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants