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

ElasticSearch shell script fails due to unsupported syntax on non-Bash shells #3691

Closed
brian-from-fl opened this Issue Sep 13, 2013 · 0 comments

Comments

Projects
None yet
2 participants
@brian-from-fl
Copy link

brian-from-fl commented Sep 13, 2013

The first line of the elasticsearch script specifies /bin/sh, but this is generally defined as pointing to a Bourne-compatible shell. It cannot be assumed to point to Bash shell.

Line 92 contains the followin:

JAVA=$(which java)

  1. The first line should be changed to /bin/bash if that is intended.
  2. Or line 92 must be changed so that which java is surrounded within backtics, which is the only common (though not able to be nested) syntax across all Bourne-derived shells (Bourne, Dash, Korn, Bash).

Thank you.

Brian Yoder

@ghost ghost assigned spinscale Sep 16, 2013

@spinscale spinscale closed this in e0af9e2 Sep 16, 2013

spinscale added a commit that referenced this issue Sep 16, 2013

Making debian init script bourne shell compatible
Using backticks instead of bash only $() to execute a shell command

Closes #3691

mute pushed a commit to mute/elasticsearch that referenced this issue Jul 29, 2015

Making debian init script bourne shell compatible
Using backticks instead of bash only $() to execute a shell command

Closes elastic#3691
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.