Skip to content

Publish macOS Intel (darwin-x86_64) release binaries - #8

Merged
alexeagle merged 1 commit into
masterfrom
figma-release-darwin-x86_64
Jun 26, 2026
Merged

Publish macOS Intel (darwin-x86_64) release binaries#8
alexeagle merged 1 commit into
masterfrom
figma-release-darwin-x86_64

Conversation

@alexeagle

Copy link
Copy Markdown

What

Adds macOS Intel (darwin-x86_64) binaries to the Figma Bazel release pipeline, matching how bazelbuild/bazel names its release assets.

The build-darwin job is converted into a matrix (mirroring the existing build-linux job), so darwin now builds both architectures:

platform runner bazelisk asset published asset
darwin-arm64 macos-14 (Apple silicon) bazelisk-darwin-arm64 bazel-<ver>-darwin-arm64
darwin-x86_64 (new) macos-13 (Intel) bazelisk-darwin-amd64 bazel-<ver>-darwin-x86_64

Both bazel-* and bazel_nojdk-* flavors plus .sha256 sidecars are produced for the new arch.

Notes

  • Bazelisk's darwin download token for x86_64 is amd64 (same quirk as linux), so the matrix carries bazelisk_arch separately from the published arch.
  • Attestation (subject-path: dist/bazel-*) and release (gh release create … dist/*) already glob over all artifacts, and the release job's needs already blocks on every matrix leg — so no other changes were required.
  • Added an inline comment flagging that macos-13 is GitHub's last Intel hosted runner and is on a deprecation track; once retired, the x86_64 leg will need to cross-compile from an Apple-silicon runner.

🤖 Generated with Claude Code

Convert the single-arch build-darwin job into a matrix mirroring
build-linux, adding an Intel macos-13 runner alongside the existing
Apple-silicon macos-14 build. Assets follow Bazelisk's <os>-<arch>
naming (bazel-<version>-darwin-x86_64, plus the bazel_nojdk flavor and
.sha256 sidecars), matching upstream bazelbuild/bazel.

Bazelisk's darwin download token for x86_64 is amd64 (as on linux), so
the matrix carries bazelisk_arch separately from the published arch.

Attestation and release steps already glob over dist/*, and the release
job's needs already blocks on all matrix legs, so no other changes are
required.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

@alexeagle alexeagle left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

LGTM thanks claude

@alexeagle
alexeagle merged commit 16a307a into master Jun 26, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant