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

Unsupported git config #1068

Closed
lhunath opened this issue Nov 9, 2021 · 5 comments · Fixed by #1140
Closed

Unsupported git config #1068

lhunath opened this issue Nov 9, 2021 · 5 comments · Fixed by #1140

Comments

@lhunath
Copy link

lhunath commented Nov 9, 2021

Environment

sentry-cli 1.66.0

in .git/config:

[extensions]
        worktreeConfig = true

Steps to Reproduce

  1. sentry-cli releases set-commits [email protected]+44 --commit "Spectre / Spectre iOS@build/43-public..HEAD"

Expected Result

The release has commits associated to it.

Actual Result

error: unsupported extension name extensions.worktreeconfig; class=Repository (6)
@kamilogorek
Copy link
Contributor

kamilogorek commented Jan 12, 2022

I just verified it locally and cannot reproduce this issue:

osx version 12.0.1
git version 2.34.1
sentry-cli version 1.66.0 (tested on 1.71.0 as well)

Can you run this command with SENTRY_LOG_LEVEL=debug and provide the output?

@lhunath
Copy link
Author

lhunath commented Jan 12, 2022

Doesn't produce any extra output. Same with debug/trace.

$ SENTRY_LOG_LEVEL=trace sentry-cli --log-level trace releases set-commits [email protected]+44 --commit "Spectre / Spectre iOS@build/43-public..HEAD"
error: unsupported extension name extensions.worktreeconfig; class=Repository (6)
$ sentry-cli -V
sentry-cli 1.71.0

@kamilogorek
Copy link
Contributor

We are using git2-rs crate to handle git integration, which internally uses libgit2, and it appears that this in itself has the problem with handing this config option, see: libgit2/libgit2#6044

@kamilogorek
Copy link
Contributor

@lhunath I just realized that I broke logging with the 1.71.0 release, debug log should be working in the latest :3

It appears that we will be able to fix it soon, once rust-lang/git2-rs#791 is released.

@github-actions
Copy link

This issue has gone three weeks without activity. In another week, I will close it.

But! If you comment or otherwise update it, I will reset the clock, and if you label it Status: Backlog or Status: In Progress, I will leave it alone ... forever!


"A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀

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

Successfully merging a pull request may close this issue.

3 participants