1 SC2175

koalaman edited this page 2016-01-25 19:43:11 -08:00

Table of Contents

• Quote this invalid brace expansion since it should be passed literally to eval
• Problematic code:
• Correct code:
• Rationale:
• Exceptions:

Quote this invalid brace expansion since it should be passed literally to eval

Problematic code:

eval echo {1..$n}

Correct code:

eval "echo {1..$n}"

Rationale:

Using eval somecommand {1..$n} depends both on bash silently failing to interpret the brace expansion, and on it passing failing brace expansions literally.

Rather than depending on these questionable features (which already behave differently in other shells), use the explicit, predictable way of passing values literally: quoting.

Exceptions:

None.

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.