Skip to content

Conversation

@apache-hb
Copy link
Contributor

adds tracing.cc to absl_base_sources, and adds tests for base, numeric, hash, crc32, container, and logging to try and catch any other missing sources.

@apache-hb apache-hb force-pushed the abseil-cpp branch 9 times, most recently from e405feb to dca64b2 Compare October 6, 2025 01:28
@apache-hb apache-hb marked this pull request as ready for review October 6, 2025 21:42
@apache-hb apache-hb force-pushed the abseil-cpp branch 2 times, most recently from 4b22b41 to ed7bd44 Compare November 2, 2025 19:39
@apache-hb apache-hb requested a review from neheb December 13, 2025 15:14
Copy link
Member

@nirbheek nirbheek left a comment

Choose a reason for hiding this comment

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

LGTM overall

@nirbheek
Copy link
Member

Will merge in a week if there are no objections.

@bonzini
Copy link

bonzini commented Dec 24, 2025

Looks good, with just an aesthetic comment that you can disregard too.

@nirbheek
Copy link
Member

clang-cl arm64 job failure seems to be related.

MSYS2 CLANGARM64 job failure needs investigating

@apache-hb
Copy link
Contributor Author

Seems the msys2 failure is due to msys2/MSYS2-packages#2058, I would guess the msys2/setup-msys2 action needs to updated to resolve that failure.

@apache-hb apache-hb requested a review from nirbheek December 31, 2025 19:31
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.

4 participants