Skip to content

fix(ci): tritrpc-exec-conformance — build crate at rust/tritrpc_v1#37

Merged
mdheller merged 2 commits into
mainfrom
fix/tritrpc-conformance-crate-path
Jul 5, 2026
Merged

fix(ci): tritrpc-exec-conformance — build crate at rust/tritrpc_v1#37
mdheller merged 2 commits into
mainfrom
fix/tritrpc-conformance-crate-path

Conversation

@mdheller

@mdheller mdheller commented Jul 5, 2026

Copy link
Copy Markdown
Member

Repairs a conformance gate that has failed on every run since 2026-04-26 (4/4 failures).

Root cause: the workflow runs cargo build -p tritrpc_v1 from the TriTRPC repo root, but the pinned commit (5874124) has no root Cargo.toml — the crate lives at rust/tritrpc_v1/ (bin src/bin/trpc.rs). Unrelated to the dependabot checkout bump that surfaced it.

Fix: build inside the crate dir; introduce TRPC_BIN env pointing at rust/tritrpc_v1/target/debug/trpc; exec step uses TRPC_BIN. Fixture paths (fixtures/vectors_hex_unary_rich.txt*) verified correct at the pin — untouched.

This PR touches the workflow file, so the path filter triggers the gate on this PR itself — green checks here = the gate actually works.

🤖 Generated with Claude Code

mdheller added 2 commits July 5, 2026 17:00
…ate never passed)

The pinned TriTRPC commit has no root Cargo.toml; the crate lives at rust/tritrpc_v1
(bin src/bin/trpc.rs). Build in the crate dir and point the exec step at the real
binary via TRPC_BIN. Fixtures paths were already correct. This gate has failed on
every run since 2026-04-26; this makes it a real conformance check.
tools/run-tests and siblings were committed without the executable bit, so the
conformance workflow failed with exit 126 the moment the build step was fixed.
@mdheller mdheller merged commit e5fa538 into main Jul 5, 2026
5 checks passed
@mdheller mdheller deleted the fix/tritrpc-conformance-crate-path branch July 5, 2026 21:10
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.

1 participant