-
-
Notifications
You must be signed in to change notification settings - Fork 605
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
key_bindings.ron should follow symlink #1751
Comments
It's a hacky solution for now, but couldn't you use hard-links ? ln ~/dotfiles/git/gitui/keyconfig.ron ~/.config/gitui/keyconfig.ron (yes it's just killing the -s soft-link flag) I agree that it should be able to follow symlinks just but it might help you for now I hope 🙂 |
Should be easy to fix using https://doc.rust-lang.org/std/fs/fn.canonicalize.html on the file-path. should be unittestable |
Hey! Is this issue already assigned to someone? If not, I would like to work on it. |
We don’t assign. You make a PR and that’s it :) |
Thanks. Will do :) |
Gitui doesn't support the soft link, we can only utilize hard link to make it work before a fixed version v0.24.2. We lock version to prevent a breaking change in the future. Refs: - gitui-org/gitui#1751 - https://github.com/extrawurst/gitui/releases/tag/v0.24.2
Describe the bug
When I use symlink to configure key bindings like below, it didn't work.
When I use
cp
instead ofln -s
it worked.To Reproduce
Steps to reproduce the behavior:
key_bindings.ron
somewhere (say,~/key_bindings.ron
)ln -s ~/keyconfig.ron ~/.config/gitui/keyconfig.ron
Expected behavior
Symlink should work.
Context (please complete the following information):
Additional context
I tried to use symlink since I wanted to put my config in my dotfiles. This could be pretty common.
The text was updated successfully, but these errors were encountered: