SC2158 – ShellCheck Wiki

See this page on GitHub

Sitemap


[ false ] is true. Remove the brackets

Problematic code:

if [ false ]
then
  echo "triggers anyways"
fi

Correct code:

if false
then
  echo "never triggers"
fi

Rationale:

[ str ] checks whether str is non-empty. It doesn't matter if str is false, it will still be evaluated for non-emptyness.

Instead, use the command false which -- as the manual puts it -- does nothing, unsuccessfully.

Exceptions:

None


ShellCheck is a static analysis tool for shell scripts. This page is part of its documentation.