1
0
Fork 0
mirror of synced 2024-06-28 11:10:46 +12:00

Replaced $(...) instead of legacy backticked ...

Backtick command substitution `...` is legacy syntax with several
issues.

1. It has a series of undefined behaviors related to quoting in POSIX.
2. It imposes a custom escaping mode with surprising results.
3. It's exceptionally hard to nest.

$(...) command substitution has none of these problems, and is therefore
strongly encouraged.
This commit is contained in:
DaYea Yim 2019-10-29 07:41:33 +00:00
parent d5f546fd38
commit 4a7237a34d

View file

@ -20,7 +20,7 @@ function setEnvironmentVariable() {
}
# Grep for variables that look like MySQL (APP_)
for _curVar in `env | grep _APP_ | awk -F = '{print $1}'`;do
for _curVar in $(env | grep _APP_ | awk -F = '{print $1}');do
# awk has split them by the equals sign
# Pass the name and value to our function
setEnvironmentVariable ${_curVar} ${!_curVar}
@ -30,4 +30,4 @@ echo newrelic.license = \"$_APP_NEWRELIC_KEY\" > /etc/php/7.3/fpm/conf.d/newreli
echo newrelic.license = \"$_APP_NEWRELIC_KEY\" > /etc/php/7.3/cli/conf.d/newrelic.ini
# Start supervisord and services
/usr/bin/supervisord -n -c /etc/supervisord.conf
/usr/bin/supervisord -n -c /etc/supervisord.conf