-
Notifications
You must be signed in to change notification settings - Fork 64
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
Apple Silicon support #165
Comments
Probably it's just a matter of changing these lines Cake_Git/src/Cake.Git/Cake.Git.csproj Lines 21 to 22 in fce1d5e
to <PackageReference Include="LibGit2Sharp" Version="0.27.0-preview-0182" />
<PackageReference Include="LibGit2Sharp.NativeBinaries" Version="2.0.315" /> Would you like to take stab at creating a PR with the above changes? |
Happily. |
Hmm, just changing those two lines doesn't seem to make my tests pass. I'm getting this error and I don't understand what it means:
|
Could you create a draft PR? So we can see if it fails on CI too? Might be that we need to add some runtime folders. Worst case we'll need upcoming Cake 2.3 as it will have native library improvements. |
Here you go, I think you need to manually run the CI for me. |
Yep OSX CI is also dying. Same error. Any ideas? |
Version 4 introduced a separate addin for Cake Frosting (Cake.Frosting.Git) which references LibGit2Sharp instead of repackaging it and therefore should also work with Silicon chips. We should keep this issue open to track progress for Cake Scripting using Cake.Git addin. |
Hey friends,
So on OSX, on the new Apple Silicon chips (M1 / M2) etc, there is a bug that libgit2sharp fixed here:
https://github.com/libgit2/libgit2sharp/pull/1955/files
Discussion is here:
libgit2/libgit2sharp#1955
I'm having trouble using Cake_Git on my M2 Macbook Air, and it seems like simply updating libgit2sharp would be sufficient to resolve the issue. But I'm not smart enough to figure this out on my own.
LibGit2Sharp 0.27.0-preview-0182 includes this change, so it should just be a matter of:
and then it's all fixed. Hopefullly.
I'm happy to help test fixes with my macbook.
The text was updated successfully, but these errors were encountered: