Ignore shellcheck warning

String replacements in bash aren't as powerful as in sed. https://stackoverflow.com/questions/13710806/string-replace-this-shell-variable http://tldp.org/LDP/abs/html/string-manipulation.html
mzagrabe · Feb 15, 2017 · 25691c4 · 25691c4
1 parent 1a1fbdf
commit 25691c4
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/download-itunes-cover b/download-itunes-cover
@@ -8,8 +8,10 @@ if test "z$URL" = "z" ; then
     exit 1
 fi
 
-BASE_URL="$(echo $URL | sed -e 's/[0-9]*x[0-9]*[^.]*\.[a-z]*$//')"
-SUFFIX="$(echo $URL | sed -e 's/^.*[0-9]*x[0-9]*\([^.]*\)\.\([a-z]*\)$/\1.\2/')"
+# shellcheck disable=SC2001
+BASE_URL="$(echo "$URL" | sed -e 's/[0-9]*x[0-9]*[^.]*\.[a-z]*$//')"
+# shellcheck disable=SC2001
+SUFFIX="$(echo "$URL" | sed -e 's/^.*[0-9]*x[0-9]*\([^.]*\)\.\([a-z]*\)$/\1.\2/')"
 
 test_size() {
     size=$1