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

bashism in sshx.sh #274

Closed
juliantaylor opened this issue Feb 16, 2011 · 2 comments
Closed

bashism in sshx.sh #274

juliantaylor opened this issue Feb 16, 2011 · 2 comments
Milestone

Comments

@juliantaylor
Copy link
Contributor

the default sshx.sh in 0.10.1 and lower uses a bash feature in a shell which may not support it:

#!/bin/sh
"$@" &> /dev/null &
echo $!

the non-posix redirection &> may not be supported when /bin/sh does not support it, e.g. dash.
a possible fix is:
"$@" > /dev/null 2>&1 &

@fperez
Copy link
Member

fperez commented Mar 13, 2011

Thanks for the report and fix! Committed locally, this will close when I push.

@fperez
Copy link
Member

fperez commented Mar 13, 2011

Remove bash-ism and use standard posix redirect in sshx.sh script.

Closed by a63739a, thanks to github user juliantaylor for the fix!

This issue was closed.
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