Skip to content

fix: fall back to amd64 h2spec on Apple Silicon#53

Merged
shaneutt merged 1 commit intopraxis-proxy:mainfrom
usize:fix/h2spec-darwin-arm64
Apr 18, 2026
Merged

fix: fall back to amd64 h2spec on Apple Silicon#53
shaneutt merged 1 commit intopraxis-proxy:mainfrom
usize:fix/h2spec-darwin-arm64

Conversation

@usize
Copy link
Copy Markdown
Member

@usize usize commented Apr 18, 2026

Summary

  • make tools fails on Apple Silicon (darwin/arm64) because h2spec v2.6.0 has no arm64 release — the download URL h2spec_darwin_arm64.tar.gz does not exist
  • This causes the h2spec_strict_conformance test to fail since the binary is never installed
  • Adds an H2SPEC_ARCH variable that falls back to amd64 on darwin_arm64, since the x86_64 binary runs fine under Rosetta 2

Test plan

  • Verified make tools constructs the correct download URL (h2spec_darwin_amd64.tar.gz) on an arm64 Mac
  • Verified SHA256 checksum matches
  • Verified h2spec binary runs correctly under Rosetta 2
  • Verified h2spec_strict_conformance test passes

@shaneutt shaneutt self-assigned this Apr 18, 2026
@github-project-automation github-project-automation Bot moved this to Backlog in AI Gateway Apr 18, 2026
@shaneutt shaneutt moved this from Backlog to Review in AI Gateway Apr 18, 2026
@shaneutt shaneutt added this to the v0.5.0 milestone Apr 18, 2026
shaneutt
shaneutt previously approved these changes Apr 18, 2026
@shaneutt shaneutt dismissed their stale review April 18, 2026 11:21

The merge-base changed after approval.

h2spec v2.6.0 has no arm64 release. On darwin/arm64, `make tools`
tries to download `h2spec_darwin_arm64.tar.gz` which does not exist,
causing the conformance test suite to fail.

Use the amd64 binary instead, which runs under Rosetta 2.
@shaneutt shaneutt force-pushed the fix/h2spec-darwin-arm64 branch from 37e50d7 to b366399 Compare April 18, 2026 11:22
@shaneutt shaneutt merged commit 391bd6a into praxis-proxy:main Apr 18, 2026
12 checks passed
@github-project-automation github-project-automation Bot moved this from Review to Done in AI Gateway Apr 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants