wfxr / forgit

:zzz: A utility tool powered by fzf for using git interactively.
MIT License
4.33k stars 136 forks source link

[Fish] ga/gd no longer works after plugin upgrade #224

Closed ray-x closed 1 year ago

ray-x commented 1 year ago

Check list

Environment info

Problem / Steps to reproduce

in a git repo, run ga or gd The script complains about forgit::extract_file not found/syntax error.

image

Update: my $SHELL is set to bash, but my current shell is fish, set $SHELL to fish works fine. Similar to this https://github.com/wfxr/forgit/issues/168

cjappl commented 1 year ago

hmm, this is interesting.

Is there a reason that your $SHELL is bash instead of fish?

See also this ticket: https://github.com/wfxr/forgit/pull/169

There was a fix for it then, but it broke mine when my $SHELL was fish...

cjappl commented 1 year ago

I'm not sure there is a long term solution that will fix both, as fish syntax is very different than bash syntax, and we rely on fish scripting internally to have it work properly

ray-x commented 1 year ago

If login shell is fish, I think it should be OK. But if you switch between shells, might be an issue.

stale[bot] commented 1 year ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

ray-x commented 1 year ago

as there is a workaround, close it for now.