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

Update libgit2 #422

Merged
merged 5 commits into from
May 17, 2022
Merged

Update libgit2 #422

merged 5 commits into from
May 17, 2022

Conversation

Jake-Shadle
Copy link
Member

@Jake-Shadle Jake-Shadle commented May 17, 2022

  • Replace structopt with clap 3
  • Remove rustsec git dependency

Resolves: #380
Resolves: #410

Rustsec has been really slow updating to a version with git 0.14 to
resolve a segfault, which also blocked cargo-deny. Instead, we now
handle all git fetching ourselves so we are no longer blocked, now or in
the future. This also changes it so that we use crates-index as a
dependency as previously we had included this functionality directly as
at the time crates-index didn't support bare repositories.
@Jake-Shadle Jake-Shadle changed the title Cleanup Update libgit2 May 17, 2022
@Jake-Shadle Jake-Shadle merged commit 84e3357 into main May 17, 2022
@Jake-Shadle Jake-Shadle deleted the upgrade/git2 branch May 17, 2022 13:39
@@ -133,19 +133,6 @@ jobs:
- name: cargo publish
run: cargo publish --dry-run

msrv-check:
name: Minimum Stable Rust Version Check

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is still useful even if rust-version is set, right?

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.

Update libgit2 Segfault due to libgit2 shenanigans
2 participants