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

feat: vendor libgit2 (via git2go) #266

Merged
merged 5 commits into from
Mar 8, 2022
Merged

feat: vendor libgit2 (via git2go) #266

merged 5 commits into from
Mar 8, 2022

Conversation

patrickdevivo
Copy link
Contributor

@patrickdevivo patrickdevivo commented Mar 7, 2022

Uses the instructions here (main branch, or vendored static linking) to vendor libgit2 and git2go into this repo.

We're encountering an issue with homebrew installations when the upstream libgit2 gets an update and we haven't upgraded mergestat to support the new libgit2 version. Homebrew doesn't seem to allow us to pin to a particular version, so installations begin breaking.

I see two possible solutions:

  1. Vendor libgit2 (and git2go) into this repo so that builds will statically link it, no need to depend on a system libgit2 (what this PR does)
  2. Update the homebrew installation to just copy a pre-built binary

Or maybe both...

@codecov
Copy link

codecov bot commented Mar 7, 2022

Codecov Report

Merging #266 (4395283) into main (1fe8e56) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #266   +/-   ##
=======================================
  Coverage   71.04%   71.04%           
=======================================
  Files          55       55           
  Lines        3119     3119           
=======================================
  Hits         2216     2216           
  Misses        574      574           
  Partials      329      329           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3c0eddf...4395283. Read the comment docs.

@patrickdevivo patrickdevivo marked this pull request as ready for review March 8, 2022 00:59
@patrickdevivo patrickdevivo merged commit 7f969dc into main Mar 8, 2022
@patrickdevivo patrickdevivo deleted the vendor-libgit2 branch March 8, 2022 01:04
@Alessandro-Barbieri
Copy link

Isn't possible to let the use choose if use the vendored or the system libgit2?

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.

2 participants