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

Incorrect Shebang in the leetmouse_manage Script #11

Closed
psyleft opened this issue Feb 17, 2022 · 1 comment
Closed

Incorrect Shebang in the leetmouse_manage Script #11

psyleft opened this issue Feb 17, 2022 · 1 comment

Comments

@psyleft
Copy link

psyleft commented Feb 17, 2022

The shebang in leetmouse_manage is #!/bin/sh.

This works fine on systems in which /bin/bash is symlinked to /bin/sh. However, if this is not the case the script raises issue with the lines

if [ $1 == "unbind_all" ]; then

and

if [ $1 == "bind_all" ]; then

The shebang should be changed to #!/bin/bash or the double equals sign == should be changed to a single equals sign =.

@psyleft
Copy link
Author

psyleft commented Feb 17, 2022

The second option is probably preferable as to not add another dependency (though it's bash, so who really cares). That being said, I'm not the best with shell scripting so there might be a better way to solve this issue.

systemofapwne added a commit that referenced this issue May 19, 2022
systemofapwne added a commit that referenced this issue May 20, 2022
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

No branches or pull requests

2 participants