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

julia build problems with libgit2 1.4.2 or 1.5.0 #44811

Closed
domr2695 opened this issue Mar 31, 2022 · 5 comments · Fixed by #45411
Closed

julia build problems with libgit2 1.4.2 or 1.5.0 #44811

domr2695 opened this issue Mar 31, 2022 · 5 comments · Fixed by #45411
Labels
building Build system, or building Julia or its dependencies

Comments

@domr2695
Copy link

domr2695 commented Mar 31, 2022

Even julia 1.9.0-DEV supports libgit2 v 1.3.0 from Sep 27, 2021.
Is it possible to support libgit2 v 1.4.2 from Feb 26, 2022 ?
Or maybe even libgit2 v 1.5.0 from Feb 26, 2022 commit:db4461d556ddb7915349d094ba74ef01d7a2d701.
Otherwise, the julia + libgit2 v 1.4.2 or v 1.5.0 build never completes.

@vtjnash
Copy link
Member

vtjnash commented Mar 31, 2022

I think there is relevant discussion here: #42311, where we have been waiting for libssh2 to make a security-critical release for several months

@domr2695
Copy link
Author

domr2695 commented Mar 31, 2022

I'm building julia 1.9.0 DEV normally with:

libssh2-1.10.0
# 2022-03-21 5:11 PM GMT+3
LIBSSH2_SHA1=e7e1312b0cbfa643e2f8bf5f2036ce5147ed797d

mbedtls-2.28.0

curl-7.82.0

libgit2-1.3.0

I think it's not about libssh2. Otherwise julia wouldn't be building fine with libgit2-1.3.0.

@ViralBShah
Copy link
Member

I believe we are already vendoring the libssh2 security patch ourselves. Maybe someone just needs to provide a PR to bump libgit2 (and work through the issues that show up)?

@fxcoudert
Copy link
Contributor

From the libgit2 release notes for 1.4.0:

Deprecated APIs

  • git_index_checksum is deprecated; this information is now internal to the library and there is no replacement
  • git_indexer_hash is deprecated; callers should use git_indexer_name to retrieve the filename
  • git_packbuilder_hash is deprecated; callers should use git_packbuilder_name to retrieve the filename

ABI changes

  • git_fetch_options now includes the follow_redirects value
  • git_push_options now includes the follow_redirects value
  • git_status_options now includes the rename_threshold value
  • git_transport contains several changed function pointer signatures

From a quick check of the julia codebase, I think the git_fetch_options and git_push_options types need to change. I would be willing to try to add these new types, if you can give some guidance.

@ViralBShah ViralBShah added the building Build system, or building Julia or its dependencies label May 21, 2022
@fxcoudert
Copy link
Contributor

I've opened a PR to fix the issue: #45411

@giordano giordano linked a pull request May 21, 2022 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
building Build system, or building Julia or its dependencies
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants