Skip to content

[CI] Cross-compile Intel Mac release binary#3078

Merged
fhanau merged 1 commit into
mainfrom
felix/110724-ci-overhaul
Nov 7, 2024
Merged

[CI] Cross-compile Intel Mac release binary#3078
fhanau merged 1 commit into
mainfrom
felix/110724-ci-overhaul

Conversation

@fhanau
Copy link
Copy Markdown
Contributor

@fhanau fhanau commented Nov 7, 2024

Follow-up to #2958, where we added and tested cross-compilation support. This allows us to use Xcode 16 to compile the Intel Mac binary (macos-15 is Apple Silicon-based, macos-13 only has Xcode 15 available) and take advantage of the faster M1-based runners.

  • Don't set minimum macOS version for host configuration, host tools are only needed during the build and do not need to be distributed beyond that. This fixes a misleading warning.

==============

Should save some precious macOS runner time. In a follow-up, we will also compile the macOS arm64 binary on GitHub CI, and eventually the Linux arm64 binary. Then all release binaries will be compiled here and we'll be able to automate the NPM publish and NPM types jobs so that releases are fully automated.

@fhanau fhanau requested a review from npaun November 7, 2024 17:37
@fhanau fhanau requested review from a team as code owners November 7, 2024 17:37
@fhanau fhanau requested a review from mar-cf November 7, 2024 17:37
@fhanau
Copy link
Copy Markdown
Contributor Author

fhanau commented Nov 7, 2024

As with all release-only changes, there is a small risk that this breaks the build, in which case I will promptly fix or revert.

Follow-up to #2958, where we added and tested cross-compilation support. This
allows us to use Xcode 16 to compile the Intel Mac binary (macos-15 is Apple
Silicon-based, macos-13 only has Xcode 15 available) and take advantage of the
faster M1-based runners.

- Don't set minimum macOS version for host configuration, host tools are only
  needed during the build and do not need to be distributed beyond that. This
  fixes a misleading warning.
@fhanau fhanau force-pushed the felix/110724-ci-overhaul branch from e8eaf5b to 01cbfa2 Compare November 7, 2024 17:57
@fhanau fhanau requested a review from a team as a code owner November 7, 2024 17:57
@fhanau fhanau merged commit 20d994e into main Nov 7, 2024
@fhanau fhanau deleted the felix/110724-ci-overhaul branch November 7, 2024 18:36
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.

3 participants