Skip to content

Support building with Protobuf version 6.x#249

Merged
jp-pino merged 5 commits intomasterfrom
jp-pino/fix-build-protobuf-6
Feb 25, 2026
Merged

Support building with Protobuf version 6.x#249
jp-pino merged 5 commits intomasterfrom
jp-pino/fix-build-protobuf-6

Conversation

@jp-pino
Copy link
Contributor

@jp-pino jp-pino commented Feb 25, 2026

This pull request introduces support for building the project on both Linux and macOS via GitHub Actions and updates the CMake configuration to handle compatibility with Protobuf 6.x and above. The most significant changes are the addition of new CI build jobs for multiple platforms and conditional linking of the Abseil library when using newer versions of Protobuf.

CI/CD Improvements:

  • Added new GitHub Actions jobs to build the project on both ubuntu-latest and macos-latest, including steps to install dependencies and build the project using CMake and Make. (.github/workflows/ci-build.yaml)

Build System Compatibility:

  • Updated CMakeLists.txt to print the Protobuf version and, if Protobuf 6.x or higher is detected, require and link the Abseil libraries to ensure compatibility. (CMakeLists.txt)

@jp-pino jp-pino self-assigned this Feb 25, 2026
@jp-pino jp-pino added the enhancement New feature or request label Feb 25, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds support for building the project with Protobuf version 6.x or higher, which requires linking against Abseil libraries. The changes include updates to the CMake configuration to detect and link Abseil when Protobuf 6.x+ is used, and additions of CI jobs to build on Linux, macOS, and a custom container environment.

Changes:

  • Added Protobuf version detection and conditional Abseil library linking in CMake
  • Added three new CI build jobs for ubuntu-latest, macos-latest, and a custom container
  • Added version logging to help debug Protobuf version issues

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 5 comments.

File Description
CMakeLists.txt Added Protobuf version detection, conditional Abseil linking for Protobuf 6.x+, and version logging
.github/workflows/ci-build.yaml Added three new build jobs (Linux, macOS, container) to test builds across platforms

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@jp-pino jp-pino merged commit 4aec8fd into master Feb 25, 2026
4 checks passed
@jp-pino jp-pino deleted the jp-pino/fix-build-protobuf-6 branch February 25, 2026 12:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants