Skip to content

feat: adopt pure-Rust protox compilation and refresh outdated dependencies - #93

Open
johannschiffer wants to merge 1 commit into
tailcallhq:mainfrom
johannschiffer:feat/protox-deps-upgrade
Open

johannschiffer wants to merge 1 commit into
tailcallhq:mainfrom
johannschiffer:feat/protox-deps-upgrade

Conversation

@johannschiffer

@johannschiffer johannschiffer commented Oct 24, 2026 •

Copy link
Copy Markdown

Summary

This PR addresses issue #44 by refreshing outdated dependencies across the dependency graph and upgrading the protobuf compilation pipeline in build.rs to use pure-Rust protox.

Key Changes

  1. Pure-Rust Protobuf Compilation (protox):
    • Replaced host protoc reliance with protox (v0.7.1).
    • Compiles news.proto, encodes the descriptor set into news_descriptor.bin in OUT_DIR for gRPC server reflection, and compiles gRPC bindings via tonic_build::configure().skip_protoc_run().
    • Completely eliminates build failures on systems/containers without a pre-installed protoc binary.
  2. Dependency Refresh:
    • Updated outdated semver-compatible dependencies across Cargo.lock.
    • Maintained clean compatibility with standard Rust toolchains (rustc 1.88+).
  3. Verification:
    • cargo test passes cleanly (tests/ci.rs -> test generate ... ok).
    • Verified in zero-protoc environments (PROTOC=/nonexistent/protoc cargo test -> PASS).

Fixes #44
/claim #44

…encies

- Integrate protox build-dependency to eliminate external protoc requirement
- Generate news_descriptor.bin and gRPC service definitions via protox + tonic_build skip_protoc_run
- Refresh Cargo.lock dependencies to latest compatible versions
- Fixes tailcallhq#44
- /claim tailcallhq#44
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.

Update all outdated dependencies

1 participant