3 SC1026

Joachim Ansorg edited this page 2021-11-12 10:29:13 +01:00

Table of Contents

• If grouping expressions inside [[..]], use ( .. ).
• Problematic code:
• Correct code:
• Rationale:
• Exceptions:
• Related resources:

If grouping expressions inside [[..]], use ( .. ).

Problematic code:

[[ [ a || b ] && c ]]
[ [ a -o b ] -a c ]]

Correct code:

[[ ( a || b ) && c ]]
[ \( a -o b \) -a c ]]  # or  { [ a ] || [ b ]; } && [ c ]

Rationale:

[ .. ] should not be used to group subexpressions inside [[ .. ]] or [ .. ] statements.

For [[ .. ]], use regular parentheses.

For [ .. ], either use escaped parentheses, or preferably rewrite the expression into multiple [ .. ] joined with &&, || and { ..; } groups. The latter is preferred because [ .. ] is undefined for more than 4 arguments in POSIX.

Exceptions:

None

Related resources:

• Bash Pitfalls: if [ [ a = b ] && [ c = d ] ]; then ...
• POSIX test

ShellCheck

• Installation

  • CentOS6
  • More Installation Guides

• Usage

  • Directive
  • Ignore

• Integrating and extending

  • Misc. contributions
  • Azure Pipelines
  • Buildkite
  • CircleCI
  • DevGuide
  • Integration
  • JUnit
  • Phabricator
  • TravisCI
  • GitLab CI
  • Drone
  • GitHub Actions

• Severity

• Template

Each individual ShellCheck warning has its own wiki page like SC1000. Use GitHub Wiki's "Pages" feature above to find a specific one, or see Checks.