Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Integrate protobuf #2471

Closed
wants to merge 3 commits into from
Closed

Integrate protobuf #2471

wants to merge 3 commits into from

Conversation

timbess
Copy link
Contributor

@timbess timbess commented Dec 8, 2023

This integrates protobuf into the cmake build using some modular tooling. We'll now need protoc installed locally to build, this will use that to parse the protobuf version used in the CLI and checkout/build the correct version based on what the user has locally.

Changes:

  • Make psp_build_dep always output to the top-level target's binary dir.
  • Improve error reporting in psp_build_dep so that errors aren't white text above the top level error message.
  • Workaround templating issue with external dependencies.
  • Adds Protoc integration module that uses the globally installed protoc and provides helpers for building C++ modules from them.

Depends on this PR being merged.

* Setup ClangD with Emscripten/Python.

Fixes:
* Fix boost include shadowing local headers.
…ing. We'll now need `protoc` installed locally to build, this will use that to parse the protobuf version used in the CLI and checkout/build the correct version based on what the user has locally.

Changes:
* Make psp_build_dep always output to the top-level target's binary dir.
* Improve error reporting in psp_build_dep so that errors aren't white text above the top level error message.
* Workaround templating issue with external dependencies.
* Adds Protoc integration module that uses the globally installed `protoc` and provides helpers for building C++ modules from them.
@timbess timbess closed this Dec 8, 2023
@timbess timbess deleted the integrate-protobuf branch December 8, 2023 21:27
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant