Skip to content

jj: Change to libgit2@1.7#177765

Merged
BrewTestBot merged 2 commits intoHomebrew:masterfrom
nicholasjng:jj-libgit2-change
Jul 18, 2024
Merged

jj: Change to libgit2@1.7#177765
BrewTestBot merged 2 commits intoHomebrew:masterfrom
nicholasjng:jj-libgit2-change

Conversation

@nicholasjng
Copy link
Copy Markdown
Contributor

Depend on libgit2@1.7 instead of the unpinned version. jj itself currently expects libgit2 < 1.8, so this change allows the dynamic linker to pick up the right shared library.

Addresses #177686.

New contributor, just thought I'd submit it quickly after the encouragement in #177686 (comment).

  • Have you followed the guidelines for contributing?
  • Have you ensured that your commits follow the commit style guide?
  • Have you checked that there aren't other open pull requests for the same formula update/change?
  • Have you built your formula locally with HOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <formula>, where <formula> is the name of the formula you're submitting?
  • Is your test running fine brew test <formula>, where <formula> is the name of the formula you're submitting?
  • Does your build pass brew audit --strict <formula> (after doing HOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <formula>)? If this is a new formula, does it pass brew audit --new <formula>?

@github-actions github-actions bot added the rust Rust use is a significant feature of the PR or issue label Jul 18, 2024
@nicholasjng
Copy link
Copy Markdown
Contributor Author

Needs a revision 1 line probably below the head metadata field.

Depend on libgit2@1.7 instead of the unpinned version.
jj itself currently expects libgit2 < 1.8, so this change allows
the dynamic linker to pick up the right shared library.

jj: rev bump

Signed-off-by: Rui Chen <rui@chenrui.dev>
@chenrui333
Copy link
Copy Markdown
Member

relates to #166767

Copy link
Copy Markdown
Member

@carlocab carlocab left a comment

Choose a reason for hiding this comment

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

Thanks!

@p-linnane p-linnane linked an issue Jul 18, 2024 that may be closed by this pull request
4 tasks
Copy link
Copy Markdown
Member

@carlocab carlocab left a comment

Choose a reason for hiding this comment

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

Thanks!

@github-actions
Copy link
Copy Markdown
Contributor

🤖 An automated task has requested bottles to be published to this PR.

@github-actions github-actions bot added the CI-published-bottle-commits The commits for the built bottles have been pushed to the PR branch. label Jul 18, 2024
@BrewTestBot BrewTestBot enabled auto-merge July 18, 2024 23:43
@BrewTestBot BrewTestBot added this pull request to the merge queue Jul 18, 2024
Merged via the queue into Homebrew:master with commit 68a1168 Jul 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI-published-bottle-commits The commits for the built bottles have been pushed to the PR branch. rust Rust use is a significant feature of the PR or issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

jj VCS cannot find correct libgit2

5 participants