-
Notifications
You must be signed in to change notification settings - Fork 32
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
compilation failed with libgit2 1.8.0 #213
Comments
I did make sure that |
Thanks for the report. Which version of libgit2 is this? I am guessing that gcc 14 (which was released a few days ago and already seems to be in archlinux) is using some stricter type checking, revealing a bug. I'll look into it asap. |
Thanks for the quick response. The version of libgit2 is 1:1.8.0-2. |
It looks like this is actually a break in libgit2 1.8.0: libgit2/libgit2#6793 This might explain why most distros (debian/homebrew/fedora) are staying with libgit2 1.7.2 for now. There are some discussions here: Homebrew/homebrew-core#166767 (comment) As a temporary workaround you can try to uninstall the libgit2 from your machine, in which case gert will try to download a compatible libgit2 version at install time. |
git_commit is not const anymore. Fixes #213
I downgraded libgit2 to 1.7.2 and this time gert compiled with no issue. Thank you. |
I've encountered this issue too (also under Arch Linux) and downgrading to It looks like a workaround is coming soon with #216 and release 2.0.2 which is great. Thanks for all the hard work on this and other libraries, very much appreciated. |
fyi, homebrew has shipped libgit2 1.8.0 |
Thanks. I have submitted an update for |
A patch release gert 2.1.0 is now on CRAN. |
Heads up that libgit2 may revert this again in 1.8.2: https://github.com/libgit2/libgit2/releases/tag/v1.8.2-rc1 |
I am running R on arch-based linux and cannot install the gert package. The error code is listed below. I am not sure is the r version or the compiler causing this issue, since I installed them successfully before. Thank you.
The text was updated successfully, but these errors were encountered: