-
Notifications
You must be signed in to change notification settings - Fork 169
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
Add support for darwin/arm64 (Apple Silicon M1) #310
Add support for darwin/arm64 (Apple Silicon M1) #310
Conversation
Hey @chrisgoffinet so this PR is very similar to another one merged recently: #307 For this, if you wish to generate a new github release and also have a release to packagecloud, you would want to include Line 33 in 235234a
Line 35 in 235234a
|
Thanks. I was just following the convention from prior commit. Might be a bug we don't push the linux-arm64, as thats how I missed that. I pushed up a change to include darwin-arm64 on github now. |
Yeah it is a bug - I've just created a PR for it: #311 |
@sindhusegment does this look good now? |
Hey @chrisgoffinet sorry for the delayed response on this. So we've actually got a linux-arm64 PR that we merged recently but we're having issues generating an arm64 release in CircleCI for both a github binary and a packagecloud debian release. Were you able to test to see if you can successfully build the package and release it? If you have, then this would be super helpful for us and it will move along both your release for darwin-arm64 and ours for linux-arm64 |
Hi @sindhusegment @chrisgoffinet , how are we going with this PR? What's needed to push this over the line? :) |
Hello @eddeee888 , unfortunately getting chamber to successfully build for arm64 did not get finished during @sindhusegment 's internship here. It is still on our team's roadmap for this year but I don't know 100% for sure which quarter that will get prioritized. In the meantime if whoever owns this PR can just make sure to test a local build for darwin arm64 I'm happy to get this approved and merged. Potentially in that case if comments in the code can mention that it would be nice. We can worry about automatic building and uploading to packagecloud at a later time. This way at minimum people can be unblocked if they need it. |
cc @emmy-byrne-segment so you're aware of this PR as well ^^ |
This all seems correct to me. Doing some local checks before we merge. |
Tested with a rudimentary test on an M1 machine and build output is functional and correct arch. |
Thank you so much! Does this mean the next release should have the arm64 version? If so, when would that be if you don't mind me asking? |
No description provided.