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

Brew installation of fish shell plugin failing #377

Closed
3 of 10 tasks
Trid-collab opened this issue Mar 28, 2024 · 9 comments · Fixed by #379
Closed
3 of 10 tasks

Brew installation of fish shell plugin failing #377

Trid-collab opened this issue Mar 28, 2024 · 9 comments · Fixed by #379
Assignees

Comments

@Trid-collab
Copy link

Trid-collab commented Mar 28, 2024

Check list

  • I have read through the README
  • I have the latest version of forgit
  • I have searched through the existing issues

Environment info

  • OS
    • Linux
    • Mac OS X
    • Windows
    • Others:
  • Shell
    • bash
    • zsh
    • fish

Problem / Steps to reproduce

I am new to this. Need help. I have installed forgit on my MacBook as per the installation instructions mentioned for both zsh and fish shells using brew

then pasted the following as mentioned

Fish:

~/.config/fish/config.fish:

[ -f $HOMEBREW_PREFIX/share/forgit/forgit.plugin.fish ]; and source $HOMEBREW_PREFIX/share/forgit/forgit.plugin.fish

Zsh:

~/.zshrc:

[ -f $HOMEBREW_PREFIX/share/forgit/forgit.plugin.zsh ] && source $HOMEBREW_PREFIX/share/forgit/forgit.plugin.zsh`

However I find that forgit works on my zsh shell but isn't in the fish shell
I get the following error if I run say 'glo' in the fish shell, where as it works on the zsh shell

``` ↪ git-forgit log
fish: Unknown command: /opt/homebrew/share/conf.d/bin/git-forgit
- (line 1):
function git-forgit --wraps /opt/homebrew/share/conf.d/bin/git-forgit --description 'alias git-forgit /opt/homebrew/share/conf.d/bin/git-forgit';  /opt/homebrew/share/conf.d/bin/git-forgit $argv
                                                                                                                                                   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^
in function 'git-forgit' with arguments 'log'
@sandr01d
Copy link
Collaborator

As you are our fish and macOS guru, could you take a look at @cjappl?

@cjappl
Copy link
Collaborator

cjappl commented Mar 28, 2024

Can you give the output of:

> ls $HOMEBREW_PREFIX/share/forgit/forgit.plugin.fish

As well as:

> ls $HOMEBREW_PREFIX/bin/git-forgit

@Trid-collab
Copy link
Author

Trid-collab commented Mar 29, 2024

↪ > ls $HOMEBREW_PREFIX/share/forgit/forgit.plugin.fish fish: Expected a string, but found a redirection

↪ > ls $HOMEBREW_PREFIX/bin/git-forgit fish: Expected a string, but found a redirection

@cjappl
Copy link
Collaborator

cjappl commented Mar 29, 2024

Hmm, how about

'''
echo $HOMEBREW_PREFIX
'''

Not really sure what your output means. Can you tell via some other means (opening finder and going to the location) that those two files exist?

@Trid-collab
Copy link
Author

Trid-collab commented Mar 29, 2024

Output of echo $HOMEBREW_PREFIX
/opt/homebrew

in finder both the files exist in their places

Screenshot 2024-03-29 at 8 32 17 PM Screenshot 2024-03-29 at 8 27 06 PM

@cjappl cjappl changed the title not able to make forgit work on fish shell Brew installation of fish shell plugin failing Mar 30, 2024
@cjappl
Copy link
Collaborator

cjappl commented Mar 30, 2024

Will be fixed once #379 is merged and we do a release to homebrew.

In the meantime I recommend just using the github repo directly. Thanks for reporting!!

@cjappl
Copy link
Collaborator

cjappl commented Apr 1, 2024

@Trid-collab please update forgit with brew and this should be fixed (I just checked it).

Please let me know if you still see the issue.

@Trid-collab
Copy link
Author

Trid-collab commented Apr 1, 2024 via email

@cjappl
Copy link
Collaborator

cjappl commented Apr 1, 2024 via email

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

Successfully merging a pull request may close this issue.

3 participants