シェルスクリプトの区切りについて

Question

sample.sh: 行 47: 予期しないトークン ${ECHO}' 周辺に構文エラーがあります sample.sh: 行 47: ${ECHO} "1-テスト"'

抜粋
ECHO=which echo

kugiri(){
${ECHO} "----------------------------------"
}

kugiri()
${ECHO} "1-テスト"

↑が47行目です。

やりたいことは
"----------------------------------"
1-テスト

というログを出力させたいです。

kugiri()
はこのあとも2-テスト
などを入れたい為関数にしてあります。

何か原因がわかる方、いらっしゃいますでしょうか。

Answer 1

関数を呼び出すときはカッコは必要ありません。kugiri のみでいいです。

Answer 2

変数 "ECHO" の値が正しくないためだと思います。

（現状）

ECHO=which echo

（修正）

ECHO=$(which echo)

Answer 3

修正箇所は他の方が指摘されているとおりですが、シェルスクリプトになれていないうちは、実行前にShellcheckで構文エラーなどをチェックするようにすると良いと思います。（CLIツールもあります）

https://www.shellcheck.net/

以下が実行結果です。大抵の場合はどこらへんに記述ミスがあるのかを指摘してくれるので、修正の手がかりになるはずです。あと致命的なミス(rm -rf /$hoge)などを事前に警告してくれるので、保険の意味でも役立つと思います。

Line 2:
ECHO=`which echo`
^-- SC2034: ECHO appears unused. Verify it or export it.
     ^-- SC2006: Use $(..) instead of legacy `..`.

蛇足ですが、ある程度長いスクリプトを書く場合は、GoogleのShell Script Sytle Guideを一読すると良いかもしれません。ベストプラクティスが非常によくまとまっていて参考になると思います。

http://google.github.io/styleguide/shell.xml

If you are writing a script that is more than 100 lines long, you should probably be writing it in Python instead

とか書いていて身も蓋もなかったりしますが。

Answer 4

お使いになられているシェルがbashですと、デフォルトの区切りはタブと改行とスペースになっているはずです。実際に私の環境で試してみましたが

HELLO=which bash
echo ${HELLO}
which

となりwhichだけが文字列となっています。

whichをコマンドとして実行後、値を代入するには

HELLO=$(which bash)

か

HELLO=`which bash`

のようにバッククォートでくくる必要があります。

関数の呼び出しは() は付けずに
「kugiri」だけにする必要があります。

Answer 5

ECHOに格納されている文字列をコマンド実行したい場合はバックコート（`）で囲んで
実行してみてください