Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci: build for macos arm #42

Closed
wants to merge 11 commits into from
Closed

ci: build for macos arm #42

wants to merge 11 commits into from

Conversation

onbjerg
Copy link
Member

@onbjerg onbjerg commented Apr 23, 2024

Do not merge

Ref https://github.blog/changelog/2024-01-30-github-actions-introducing-the-new-m1-macos-runner-available-to-open-source/

Builds x86 Linux and macOS binaries, as well as macOS ARM binaries in CI. This prevents users from having to build by themselves when using Homebrew unless they explicity want to.

Additionally

  • disables jemalloc on non-x86 targets, except for macOS ARM (because the page sizes play nice here)
  • enables asm-keccak unless we're on ARM linux

Closes #41

@onbjerg onbjerg added the enhancement New feature or request label Apr 23, 2024
@onbjerg onbjerg marked this pull request as ready for review April 23, 2024 23:01
@@ -12,18 +12,18 @@ class Reth < Formula
strategy :github_latest
end

bottle do
Copy link
Member Author

Choose a reason for hiding this comment

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

i removed this to have it regen the bottles

Formula/reth.rb Outdated Show resolved Hide resolved
@onbjerg onbjerg requested a review from DaniPopes April 23, 2024 23:20
sha256 cellar: :any_skip_relocation, x86_64_linux: "7b9c2d91b6ae69b5deace9d2678cec9e843bac772080bdc8191b9a49919d1111"
end

depends_on "llvm" => :build
Copy link
Member Author

Choose a reason for hiding this comment

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

i removed this because it was actually silently causing artifacts to go missing for x86 mac due to llvm already being installed by rust, just a diff version - so the linker would get confused 🤷🏻

@github-actions github-actions bot closed this in 7052490 Apr 24, 2024
@github-actions github-actions bot deleted the onbjerg/macos-arm branch April 24, 2024 09:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request pr-pull
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add macOS ARM builds
3 participants