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

Support for building refactored libgit2 sources for vendored-libgit2 feature #822

Merged

Conversation

bierbaum
Copy link
Contributor

Since libgit2/libgit2@3a3ab06 libgit2 is refactored into a library and CLI. This change allows git2-rs to build those sources when the copy is used.

This should not be integrated until a libgit2 release is made incorporating those sources. The libgit2 submodule has been updated at the current commit on libgit2's main branch.

…ed libgit2

Since libgit2/libgit2@3a3ab06 libgit2 is refactored into a library and CLI. This change allows git2-rs to build those sources when the  copy is used.

This should not be integrated until a libgit2 release is made incorporating those sources. The libgit2 submodule has been updated at the current commit on libgit2's `main` branch.
@alexcrichton
Copy link
Member

Thanks!

@alexcrichton alexcrichton merged commit b9cbb44 into rust-lang:master Mar 16, 2022
@ehuss
Copy link
Contributor

ehuss commented Mar 18, 2022

@alexcrichton Do we want to keep libgit2-sys tied to specific tagged releases? If so, this may make it difficult to make releases with changes until the next upstream release.

@alexcrichton
Copy link
Member

In general since we use pkg-config to find a system version I think we need to follow the upstream, but I figured that this was ok since there wasn't any ABI changes so whether or not the system version is used the libgit2-sys crate should still have the right bindings I think?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants