Что за ошибка в скрипте configure и как её исправить?

Question

[nihi1ist]

Пытаюсь сконфигурировать пакет, но он регулярно выдает сообщение:

строка 480: continue: имеет значение только для циклов «for», «while» или «until»

Ругается на эту строку:
if [ -z $var ]; then continue; fi

Скрипт полностью.

Подскажите, как должна выглядеть строка?

Answer 1

[Roman Ratkin]

break и continue используются только внутри циклов для управлениями ими. Если вы хотите прерывать выполнение скрипта когда $var не пуста, то как вариант

if ! [ -z $var ]; then return 1; fi

Comments

[nihi1ist]

А shellcheck предложил использовать return вместо continue. И скрипт перестал сыпать ошибками. Правильно ли такое решение?

[Roman Ratkin]

nihi1ist, а что должно произойти если $var не пуста? Функция должна не выполняться дальше или же весь скрипт должен завершиться?

[nihi1ist]

Roman Ratkin, не знаю, я вообще методом научного тыка разбираюсь. Скрипт должен сконфигурировать исходники для последующей сборки.

[Roman Ratkin]

nihi1ist, ну, раз методом тыка, то предположу, что

if ! [ -z $var ]; then return 1; fi

должно сработать. Изначально, как я понимаю, если переменная $var пуста, то мы продолжаем выполнять функцию (программу???). Соответственно, переписываем так, что если она не пуста, то завершаем функцию.

[nihi1ist]

Roman Ratkin, а сам скрипт можете посмотреть? Если есть возможность и желание конечно. Думаю так нагляднее будет?

[Roman Ratkin]

nihi1ist, я посмотрел. И, признаться кроме описанной выше конструкции идей не пришло. Вообще, очень странно, что автор этого скрипта в двух местах использовал конструкцию с continue вне цикла. А какой пакет собирает скрипт?

[nihi1ist]

Roman Ratkin, древний bomi player. Всё никак не удается собрать этот пакет для Debian Testing.

[Roman Ratkin]

nihi1ist, исправление continue на return тоже не помогло?

[nihi1ist]

Roman Ratkin, помогло. Но интересно, как вариант правильный.

[Roman Ratkin]

nihi1ist, можете спросить в ветку на гитхабе. Но, подозреваю, что они тоже самое и посоветуют. Либо, как-то кардинально перепишут этот фрагмент.

[Saboteur]

nihi1ist, Чтобы знать правильный вариант, нужно знать что собственно нужно сделать - выйти, вернуть с ошибкой, прервать выполнение. И видеть нужно чуть больше кода.

return работает с функциями, то есть этот if это часть какой-то функции?

[Roman Ratkin]

Saboteur, да, там функция. Интересно, если функция будет крутиться в цикле, то можно будет в ней использовать continue или нет?

check_feature_one() {
    local var=$(get_feature_name)
    local pkg=$(get_option_field $f_pkg)
    local value=$(eval echo \$$var)
    if [ -z $var ]; then continue; fi
    if [ $value = enable ]; then
        check_pkg_or_exit "$pkg"
    elif [ $value = autodetect ]; then
        if check_pkg "$pkg"; then
            value="enable"
            eval $var="enable"
        else
            value="disable"
            eval $var="disable"
        fi
    fi
    if [ $value = enable ]; then
    add_define HAVE_$var 1
    else
    add_define HAVE_$var 0
    fi
}

[Saboteur]

Roman Ratkin,
Нельзя.

Парсер bash в данном случае не сможет найти цикл, поскольку будет проверять только текст функции.