Skip to content

fix: make macOS release builds work on Apple Silicon - #258

Merged
emidoots merged 1 commit into
scip-ruby/masterfrom
emi/fix-release-builds
Sep 3, 2026
Merged

fix: make macOS release builds work on Apple Silicon#258
emidoots merged 1 commit into
scip-ruby/masterfrom
emi/fix-release-builds

Conversation

@emidoots

@emidoots emidoots commented Sep 3, 2026

Copy link
Copy Markdown
Member

Before this change, the macOS release job failed on the arm64 runner https://github.com/sourcegraph/scip-ruby/actions/runs/33796326759/job/100784771452:

  clang: error: include path for libstdc++ headers not found
  [-Werror,-Wstdlibcxx-not-found]

After this change, we unblock release jobs: release-mac builds for the host architecture instead, so that macOS release artifacts are genuinely arm64 (as the README and gem names claim, despite previously incorrectly being x86_64 binaries.) toolchains_llvm is patched with a backport of upstream PR 686 (released in v1.7.0) that drops -L<toolchain>/lib on macOS; we can remove this patch when we upgrade later.

Motivation

Need to release v0.4.8 #255

Test plan

Verified locally on arm64: release-mac and dev builds of //main:scip-ruby succeed, the release binary is arm64 with only system dylib deps, and //gems/scip-ruby produces both arm64-darwin gems.

Before this change, the macOS release job failed on the arm64 runner:

```
  clang: error: include path for libstdc++ headers not found
  [-Werror,-Wstdlibcxx-not-found]
```

After this change, we unblock release jobs: release-mac builds for the
host architecture instead, so that macOS release artifacts are genuinely
arm64 (as the README and gem names claim, despite previously incorrectly
being x86_64 binaries.) toolchains_llvm is patched with a backport of
upstream PR 686 (released in v1.7.0) that drops `-L<toolchain>/lib` on
macOS; we can remove this patch when we upgrade later.

Verified locally on arm64: `release-mac` and `dev` builds of `//main:scip-ruby`
succeed, the release binary is arm64 with only system dylib deps, and
`//gems/scip-ruby` produces both `arm64-darwin` gems.

Signed-off-by: Emi <emi@sourcegraph.com>

@eseliger eseliger left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

gotta love bazel for its patching stuff

@emidoots
emidoots merged commit eaa63a4 into scip-ruby/master Sep 3, 2026
3 checks passed
@emidoots
emidoots deleted the emi/fix-release-builds branch September 3, 2026 20:58
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.

2 participants