Skip to content

Commit a508e28

Browse files
fix: Ensure protoc is available in CI environments
Previous commit attempted to use protobuf-src which caused Windows build failures due to CMake/MSVC linking issues. This change reverts to requiring system-installed protoc and ensures it's installed in CI workflows. Changes: - Revert protobuf-src dependency (causes Windows linker errors) - Add protoc installation to sanitizers workflow - Other workflows already have protoc via Nix or existing install steps
1 parent 72f3a04 commit a508e28

File tree

2 files changed

+30
-35
lines changed

2 files changed

+30
-35
lines changed

.github/workflows/sanitizers.yaml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -42,6 +42,10 @@ jobs:
4242
remove_dotnet: true
4343
remove_haskell: true
4444

45+
- name: Install protoc
46+
run: sudo apt-get update && sudo apt-get install -y protobuf-compiler
47+
shell: bash
48+
4549
- name: Setup Bazel
4650
uses: >- # v0.13.0
4751
bazel-contrib/setup-bazel@663f88d97adf17db2523a5b385d9407a562e5551

Cargo.lock

Lines changed: 26 additions & 35 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)