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

safe.directory '*' is not accepted #1293

Closed
laktak opened this issue Aug 23, 2022 · 7 comments
Closed

safe.directory '*' is not accepted #1293

laktak opened this issue Aug 23, 2022 · 7 comments
Labels
bug Something isn't working help wanted Extra attention is needed upstream

Comments

@laktak
Copy link
Contributor

laktak commented Aug 23, 2022

Describe the bug

git works on mounted directories if you use

git config --global --add safe.directory '*'

But gitui will throw an error:

invalid path
please run gitui inside of a non-bare git repository

I can work around the issue by using

git config --global --add safe.directory /mnt/...

but I have to do this for every directory.

To Reproduce
Steps to reproduce the behavior:

  • run git config --global --add safe.directory '*'
  • go to a mounted directory
  • run git clone /~https://github.com/extrawurst/gitui
  • cd into gitui and run git status # this will work
  • run gitui # this will give an error

Expected behavior
gitui should use the same options as git (if this is a problem of a dependency please let me know so I can report it there)

Context (please complete the following information):

  • arch
  • gitui 0.20.1 (from arch repos)
@laktak laktak added the bug Something isn't working label Aug 23, 2022
@extrawurst
Copy link
Owner

that certainly needs to be reported in upstream: /~https://github.com/libgit2/libgit2

@laktak
Copy link
Contributor Author

laktak commented Feb 28, 2023

This has been fixed upstream.

@extrawurst
Copy link
Owner

waiting for git2-rs to cut a release for this: rust-lang/git2-rs#935

@laktak
Copy link
Contributor Author

laktak commented May 16, 2023

Appears to be ready?

@Hpmason
Copy link

Hpmason commented Jun 20, 2023

This appears to be fixed in the master branch, but the latest crates.io release still isn't fixed. For anyone still running into this issue try installing gitui from master:

cargo install --git /~https://github.com/extrawurst/gitui

@extrawurst
Copy link
Owner

Can someone validate if this works now since 0.25 should contain the upstream feature now

@extrawurst extrawurst added help wanted Extra attention is needed upstream labels Feb 27, 2024
@laktak
Copy link
Contributor Author

laktak commented Mar 29, 2024

Yes, it works without problems, thank you!

@laktak laktak closed this as completed Mar 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working help wanted Extra attention is needed upstream
Projects
None yet
Development

No branches or pull requests

3 participants