Skip to content
Joachim Ansorg edited this page Nov 12, 2021 · 7 revisions

Useless echo? Instead of cmd $(echo foo), just use cmd foo.

Problematic code:

greeting=$(echo "Hello, $name")
# or
tar czf "$(echo "$(date +%F).tar.gz")" *

Correct code:

greeting="Hello, $name"
# or
tar czf "$(date +%F).tar.gz" *

Rationale:

You appear to be using echo to write a value to stdout, and then using $(..) or `..` to capture the value again. This is as pointless as mailing yourself a postcard: you already have what you want, so there's no need to send it on a round trip.

You can just replace $(echo myvalue) with myvalue.

Exceptions

Sometimes this pattern is used because of side effect of echo or expansions. For example, here $(echo ..) is used to expand a glob.

glob="*.png"
files="$(echo $var)"

The echo is not useless, but this code is problematic because it concatenates filenames by spaces. This will break filenames containing spaces and other characters later when the list is split again. Better options are:

  • Arrays, if supported by the shell: files=( $glob ); echo "The first file is ${files[0]}"
  • Positional parameters when possible: set -- $glob; echo "The first file is $1"
  • Delaying expansion until it's needed: for file in $glob; do ...

All three methods will let you avoid issues with special characters in filenames.

As another example, here $(echo ..) is used to expand escape sequences:

unexpanded='var\tvalue'
expanded="$(echo "$var")"

In this case, use printf instead. It's well defined with regard to escape sequences.

Finally, if you really do want to concatenate a series of elements by a character like space, consider doing it explicitly with for or printf (e.g. printf '%s\n' $glob).

ShellCheck

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.

Clone this wiki locally