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

Program not installed #37

Open
KungPaoChicken opened this issue Sep 28, 2016 · 6 comments
Open

Program not installed #37

KungPaoChicken opened this issue Sep 28, 2016 · 6 comments

Comments

@KungPaoChicken
Copy link

KungPaoChicken commented Sep 28, 2016

On my Debian 8.6, with would not work on any program, giving the error: Program is not installed message. This happened on both bash and fish shells.

It seems that you missed a "$@" in the last line, and parameters are not passed to your script.

I changed the last line from main to main "$@" and things started working again. I am not sure if there are side-effects, as a result I posted this as an issue so more people could help.

@desyncr
Copy link
Contributor

desyncr commented Sep 28, 2016

Which shell are you using?

@KungPaoChicken
Copy link
Author

KungPaoChicken commented Sep 28, 2016

It was tested on both bash and fish. Issue description updated.

@desyncr
Copy link
Contributor

desyncr commented Sep 28, 2016

I can confirm issue with bash 3.2.

GNU bash, version 3.2.57(1)-release (x86_64-apple-darwin14)

@desyncr
Copy link
Contributor

desyncr commented Sep 28, 2016

Issue was introduced in this commit: 5f7e2d5#diff-23a58bf9274bedb19375e527a0744fa9R357

Effectively adding "$@" to 'main' function call solves the issue. If you'd like to PR please go ahead, otherwise I'm gonna give a shot at it.

Best regards,

@KungPaoChicken
Copy link
Author

Thanks for confirming the findings, and please go ahead with your PR.

Have a nice day.

@maturanomx
Copy link
Contributor

I just made #40 PR. I didn't see this issue 😄

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