ctdb-scripts: Avoid shellcheck warning SC2002 (useless cat)

SC2002: Useless cat. Consider 'cmd < file | ..' or 'cmd file | ..' instead. Signed-off-by: Martin Schwenke <[email protected]> Reviewed-by: Amitay Isaacs <[email protected]>
whyshan · Jul 21, 2016 · 77341f6 · 77341f6
1 parent fc858a1
commit 77341f6
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 5 deletions.
diff --git a/ctdb/config/debug_locks.sh b/ctdb/config/debug_locks.sh
@@ -32,8 +32,7 @@ loadconfig ctdb
 
     # Parse /proc/locks and extract following information
     #    pid process_name tdb_name offsets [W]
-    out=$( cat /proc/locks |
-    grep -F "POSIX  ADVISORY  WRITE" |
+    out=$( grep -F "POSIX  ADVISORY  WRITE" /proc/locks |
     awk '{ if($2 == "->") { print $6, $7, $8, $9, "W" } else { print $5, $6, $7, $8 } }' |
     while read pid rest ; do
 	pname=$(readlink "/proc/${pid}/exe")

diff --git a/ctdb/config/statd-callout b/ctdb/config/statd-callout
@@ -91,7 +91,7 @@ case "$1" in
                 printf "/statd-state@%s@/p\n", ip }')
 	# Intentional multi-word expansion for multiple files
 	# shellcheck disable=SC2086
-        if cat $files | sed -n "$sed_expr" | $CTDB ptrans "ctdb.tdb" ; then
+        if sed -n "$sed_expr" $files | $CTDB ptrans "ctdb.tdb" ; then
             rm $files
 	fi
         ;;

diff --git a/ctdb/tools/ctdb_diagnostics b/ctdb/tools/ctdb_diagnostics
@@ -106,7 +106,7 @@ show_file() {
     echo "  ================================"
     echo "  File: $fname"
     echo "  $_fdetails"
-    cat "$fname" 2>&1 | sed 's/^/  /'
+    sed 's/^/  /' "$fname" 2>&1
     echo "  ================================"
 }
 
@@ -138,7 +138,7 @@ show_and_compare_files () {
 		echo "  ================================"
 		echo "  File (on node $n): $f"
 		echo "  `onnode $n ls -l $f 2>&1`"
-		cat "$fstf" | sed 's/^/  /'
+		sed 's/^/  /' "$fstf"
 		echo "  ================================"
 		first=false
 	    else