☰

ShellCheck придирается, что мое выражение не заключено в двойные кавычки, хотя на самом деле оно там находится. Почему?

tag-icon • tag-icon bash shell-script shell shellcheck
ShellCheck придирается, что мое выражение не заключено в двойные кавычки, хотя на самом деле оно там находится. Почему?

Я пишу скрипт bash с помощью AWS CLI и shellcheckвыдает ошибку, которая, как я считаю, неверна. Я хотел бы попытаться выяснить, почему он придирается.

Вот код и сообщение об ошибке:

for server in $(${aws} ec2 describe-instances --query 'Reservations[].Instances[][].{Name: Tags[?Key==`Name`].Value[] | [0]}' --filters "Name=tag:Name,Values=${server_name}*" --output text);
                                                                                                                                                                            ^-- SC2016: Expressions don't expand in single quotes, use double quotes for that.

Я не могу правильно выстроить код в редакторе SO, но указывает ^--на *в коде. Эта часть:

"Name=tag:Name,Values=${server_name}*"

Ошибка обеспечиваетссылка на документацию ShellCheckдля справки, но когда я все перепроверил, похоже, что я соблюдаю требования. :D

Я предполагаю, что это *сбивает с толку, и я знаю, что могу обойти это, сделав это shellcheck -e SC2016, но мне действительно интересно, что может быть причиной того, что Shellcheck ведет себя некорректно.

Есть идеи?

решение1

Это ложное срабатывание, но это не то, что вы думаете. Это не имеет никакого отношения к *, и не указывало на это для меня. Это расстраивает, что `Name`находится внутри одинарных кавычек. Например, echo '`Name`'выдает то же самое предупреждение, потому что думает, что вы хотите, чтобы обратные кавычки были оценены, поэтому предупреждает вас, что они не будут оценены.

решение2

Не ответ, а отформатированный комментарий:

Педантично, вам следует использовать не forцикл, а while readпетлю:

while IFS= read -r server; do
    : do stuff here
done < <(
    "$aws" ec2 describe-instances \
        --query 'Reservations[].Instances[][].{Name: Tags[?Key==`Name`].Value[] | [0]}' \
        --filters "Name=tag:Name,Values=${server_name}*" \
        --output text
)

forциклы чтения разделены пробеламислова, whileциклы чтениялинии-- видетьhttp://mywiki.wooledge.org/BashFAQ/001

В качестве альтернативы используйте readarrayдля захвата выходных данных

readaray -t servers < <(
    "$aws" ec2 describe-instances \
        --query 'Reservations[].Instances[][].{Name: Tags[?Key==`Name`].Value[] | [0]}' \
        --filters "Name=tag:Name,Values=${server_name}*" \
        --output text
)

for server in "${servers}"; do ...; done

Наконец, для длинных и нечитаемых команд сохранение параметров в массиве может улучшить читаемость:

opts=(
    --query 'Reservations[].Instances[][].{Name: Tags[?Key==`Name`].Value[] | [0]}' 
    --filters "Name=tag:Name,Values=${server_name}*"
    --output text
)

readarray -t servers < <("$aws" ec2 describe-instances "${opts[@]}")

Связанный контент