Skip to content

GH-50637: [C++] Register bundled Abseil libraries in build_absl()#50638

Open
Reranko05 wants to merge 1 commit into
apache:mainfrom
Reranko05:arrow-java-1243
Open

GH-50637: [C++] Register bundled Abseil libraries in build_absl()#50638
Reranko05 wants to merge 1 commit into
apache:mainfrom
Reranko05:arrow-java-1243

Conversation

@Reranko05

@Reranko05 Reranko05 commented Jul 25, 2026

Copy link
Copy Markdown
Contributor

What changes are included?

Move the registration of bundled Abseil static libraries from build_protobuf() to build_absl().

Since the Abseil targets are defined in build_absl(), registering them there keeps the bundled dependency registration with the component that provides them instead of coupling it to the Protobuf build logic.

Are these changes tested?

  • Configured Arrow successfully with CMake.
  • Built Arrow successfully with the default configuration.
  • CI will verify the change across supported platforms, including the Windows configuration where this issue was originally observed.

Are there any user-facing changes?

No. This is an internal CMake/build system change.

Closes: #50637

@github-actions github-actions Bot added the awaiting review Awaiting review label Jul 25, 2026
@Reranko05

Copy link
Copy Markdown
Contributor Author

All triggered CI checks are green. If someone could trigger Windows JNI workflow, it would help verify this fix against the original issue.

@Reranko05
Reranko05 marked this pull request as ready for review July 25, 2026 14:11
Copilot AI review requested due to automatic review settings July 25, 2026 14:11

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@tadeja tadeja added the CI: Extra: C++ Run extra C++ CI label Jul 25, 2026
@Reranko05

Copy link
Copy Markdown
Contributor Author

Thanks @tadeja

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[C++] Abseil bundled static libraries are registered in build_protobuf() instead of build_absl()

3 participants