LINUX.ORG.RU

Наделают говноскрипртоты, а потом оказывается что всё равно нужны статические анализаторы как для сишко-ассемблеров.

anonymous ()

нашел еще одну странность, у меня в коде было:

kill $PID
[ $? = 0 ] && echo "killed" 

shellcheck рекомендовал сделать как:

if [ $(kill $PID) ]; then 
echo "killed"
fi
и вот этот вариант условия у меня не срабатывает.

irton ★★★★★ ()
Ответ на: комментарий от spijet

точно не помню, но вроде и такое пробовал, не помогало

у меня еще есть анализ результата

grep --quiet "send to" /etc/asound.conf
и на это bash ругается что слишком много аргументов если просто в if подставить.

irton ★★★★★ ()
Ответ на: комментарий от irton

Это как бы не эквивалентно, от слова совсем. По-моему и без линтеров должно быть очевидно, что в первом случае проверяется код выхода, а во втором что текст не пустой (причём проверяется неправильно, надо либо кавычки либо [[).

d_a ★★★★★ ()
Ответ на: комментарий от d_a

ну я привел просто пример строки, мне достаточно бы разобраться в том почему у меня не сработал первый случай.

Во втором случае мне тоже только код завершения анализировать нужно, сами данные не требуются.

irton ★★★★★ ()

Хорошая штука.

А там есть настройки? Например, возможность менять «уровень» предупреждений (например deny/warn/allow). Желательно как для всего скрипта, так и для отдельных блоков. Чтобы можно было «вкусовщину» под себя настроить и убрать false positives где они возникают. Чтение README не помогло, ставить/запускать пока не пробовал.

mironov_ivan ★★★★★ ()