1 SC2265
Vidar Holen edited this page 2022-11-02 20:15:19 -07:00

Use && for logical AND. Single & will background and return true.

Problematic code:

if [ "$1" = "install" ] & [ "$USER" != "root" ]
then
  echo "Must be root to install"
fi

Correct code:

if [ "$1" = "install" ] && [ "$USER" != "root" ]
then
  echo "Must be root to install"
fi

Rationale:

ShellCheck found a test command followed by a &. This runs the test in the background, effectively ignoring it. To specify "logical AND" between two commands, use &&.

Exceptions:

None

  • Help by adding links to BashFAQ, StackOverflow, man pages, POSIX, etc!