-
-
Notifications
You must be signed in to change notification settings - Fork 5.4k
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
[LibGit2_jll] Update to v1.6.1 #48789
Conversation
Updated and pushed |
const libgit2 = "@rpath/libgit2.1.5.dylib" | ||
const libgit2 = "@rpath/libgit2.1.6.dylib" | ||
else | ||
const libgit2 = "libgit2.so.1.5" | ||
const libgit2 = "libgit2.so.1.6" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sounds like libgit2 v1.6 has still soversion 1.5. Not sure that was done on purpose though, although it'd be great if they didn't break the ABI again.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Might have not been done on purpose, given the fact now there's v1.6.1 which includes this: libgit2/libgit2@8a871d1
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And v1.6.0 was retracted
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks like the build product is still called libgit2.so.1.5
https://github.com/JuliaBinaryWrappers/LibGit2_jll.jl/releases/tag/LibGit2-v1.6.0%2B0
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, they tagged 1.6.0 before merging libgit2/libgit2#6501, the soversion was not bumped in v1.6.0. This has been rectified in v1.6.1.
New Yggdrasil PR for 1.6.1: JuliaPackaging/Yggdrasil#6313 |
The rebase didn't go well, this PR now includes unrelated commits |
Should be fixed now |
I restarted the macOS job, second time worked like a charm. FreeBSD and PowerPC are known to have problems (and for this reason they are tier 2 or 3), and they are allowed to fail tests in CI, in fact they always fail. Thanks a lot for your work! |
Depends on JuliaPackaging/Yggdrasil#6297
Could someone remind me of the magic command to regeneratedeps/checksums/libgit2
?got it:
make -f contrib/refresh_checksums.mk libgit2