Skip to content

Temporarily restore spdlog and fmt#917

Merged
rapids-bot[bot] merged 3 commits intorapidsai:branch-25.12from
vyasr:fix/temp_restore_spdlog
Sep 30, 2025
Merged

Temporarily restore spdlog and fmt#917
rapids-bot[bot] merged 3 commits intorapidsai:branch-25.12from
vyasr:fix/temp_restore_spdlog

Conversation

@vyasr
Copy link
Copy Markdown
Contributor

@vyasr vyasr commented Sep 29, 2025

Description

Until we get a release of rapids-logger out that doesn't depend on the current spdlog behavior RAPIDS builds need these functions in place. This PR is a temporary patch to reduce the urgency of getting that rapids-logger release out.

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

@vyasr vyasr self-assigned this Sep 29, 2025
@vyasr vyasr requested review from a team as code owners September 29, 2025 23:27
@vyasr vyasr requested review from msarahan and removed request for a team September 29, 2025 23:27
@vyasr vyasr added bug Something isn't working non-breaking Introduces a non-breaking change labels Sep 29, 2025
@vyasr
Copy link
Copy Markdown
Contributor Author

vyasr commented Sep 29, 2025

/merge

@rapids-bot rapids-bot bot merged commit fdc49ad into rapidsai:branch-25.12 Sep 30, 2025
14 checks passed
@vyasr vyasr deleted the fix/temp_restore_spdlog branch September 30, 2025 04:59
vyasr added a commit that referenced this pull request Sep 30, 2025
vyasr added a commit to vyasr/rapids-cmake that referenced this pull request Sep 30, 2025
rapids-bot bot pushed a commit that referenced this pull request Oct 3, 2025
This PR reverts #917 to remove the intended functionality. The corresponding new release of rapids-logger 0.2.0 should ensure compatibility. We're moving straight to 0.2.1 because it has a bugfix we need.

Resolves rapidsai/rapids-logger#46

Authors:
  - Vyas Ramasubramani (https://github.com/vyasr)

Approvers:
  - Robert Maynard (https://github.com/robertmaynard)
  - Bradley Dice (https://github.com/bdice)

URL: #918
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working non-breaking Introduces a non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants