206

I'm writing a shell script, and I'm trying to check if the output of a command contains a certain string. I'm thinking I probably have to use grep, but I'm not sure how. Does anyone know?

1
  • Does the command need to keep running after generating the output string you're looking for, or can it be immediately closed at that time? (Your two answers differ in terms of their semantics in this respect). Mar 30, 2016 at 15:49

5 Answers 5

240

Testing $? is an anti-pattern.

if ./somecommand | grep -q 'string'; then
  echo "matched"
fi
4
  • If by any chance you only want to test a fixed string, add F and x options: grep -Fxq F stands for fixed (not interpreted) and x for the whole line
    – Erdal G.
    Dec 11, 2017 at 11:51
  • 10
    Why testing of $? is anti-pattern? Oct 9, 2018 at 18:12
  • 3
    @VitalyZdanevich I assume because it's not robust against concurrency. Oct 16, 2018 at 2:54
  • 3
    @VitalyZdanevich, for one, testing $? doesn't set the preceding commands as "checked" for purposes of set -e or the ERR trap, so your program can exit in cases where you want it to simply return down the intentionally-false path later. For another, $? is volatile global state -- it's easy to throw away its value by accident. For example, if you add a line of logging like echo "Exit status is $?", the new value in $? becomes the exit status of echo. Mar 12, 2019 at 15:40
177

Test the return value of grep:

./somecommand | grep 'string' &> /dev/null
if [ $? == 0 ]; then
   echo "matched"
fi

which is done idiomatically like so:

if ./somecommand | grep -q 'string'; then
   echo "matched"
fi

and also:

./somecommand | grep -q 'string' && echo 'matched'
8
18

Another option is to check for regular expression match on the command output.

For example:

[[ "$(./somecommand)" =~ "sub string" ]] && echo "Output includes 'sub string'"
12

A clean if/else conditional shell script:

if ./somecommand | grep -q 'some string'
then
  echo "exists"
else
  echo "doesn't exist"
fi
1
  • This seems to be a restatement of earlier answers, except the trivially obvious else addition.
    – tripleee
    May 4 at 17:08
10

SHORT ANSWER

All the above (very excellent) answers all assume that grep can "see" the output of the command, which isn't always true:

SUCCESS can be sent to STDOUT while FAILURE to STDERR.

So depending on which direction you test, your grep can fail. That's to say that if you are testing for the case of FAILURE you must redirect the output of the command to STDOUT using 2>&1 in such a case as this.

LONGER ANSWER w/ PROOFS

I had what I thought was a very simple test in a bash script using grep and it kept failing. Much head scratching followed. Use of set -x in my script revealed that the variable was empty! So I created the following test to understand how things were breaking.

NOTE: iscsiadm is a Linux tool from the "open-iscsi" package used to connect/disconnect a host to SAN storage. The command iscsiadm -m session is used to show if any LUN connections are established):

#!/bin/bash

set -x

TEST1=$(iscsiadm -m session)
TEST2=$(iscsiadm -m session 2>&1)
echo
echo 'Print TEST1'
echo $TEST1
echo
echo 'Print TEST2'
echo $TEST2
echo

If a LUN WAS connected, BOTH variables were successfully populated with values:

Print TEST1
tcp: [25] 192.168.X.XX:3260,1 iqn.2000-01.com.synology:ipdisk.Target-LUN1 (non-flash) tcp: [26] 192.168.X.XX:3260,1 iqn.2000-01.com.synology:storagehost.Target-LUN1 (non-flash)

Print TEST2
tcp: [25] 192.168.X.XX:3260,1 iqn.2000-01.com.synology:ipdisk.Target-LUN1 (non-flash) tcp: [26] 192.168.X.XX:3260,1 iqn.2000-01.com.synology:storagehost.Target-LUN1 (non-flash)

However, if a LUN WASN'T connected, iscsiadm sent the output to STDERR, and only the "TEST2" variable was populated where we had redirected to STDOUT using 2>&1; "TEST1" variable which had no redirection to STDOUT was empty:

iscsiadm: No active sessions.

Print TEST1


Print TEST2
iscsiadm: No active sessions.

CONCLUSION

If you have a funky, half-broken- works in one direction but not the other- situation such as this, try the above test replacing iscsiadm with your own command and you should get the proper visibility to rewrite your test to work correctly.

2
  • 3
    That was the missing piece I needed. Commands like resize2fs need it (2>&1). Thanks for clarifying!
    – FalloutBoy
    Dec 28, 2021 at 19:39
  • 1
    @FalloutBoy No prob; happy to help. This site has saved me tons of wasted cycles, so it seems right we all take a moment to contribute something back-
    – F1Linux
    Dec 28, 2021 at 20:06

Your Answer

Reminder: Answers generated by Artificial Intelligence tools are not allowed on Stack Overflow. Learn more

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Not the answer you're looking for? Browse other questions tagged or ask your own question.