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?
5 Answers
Testing $?
is an anti-pattern.
if ./somecommand | grep -q 'string'; then
echo "matched"
fi
-
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
-
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 ofset -e
or theERR
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 likeecho "Exit status is $?"
, the new value in$?
becomes the exit status ofecho
. Mar 12, 2019 at 15:40
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'
-
3This code doesn't work with all POSIX shells: The POSIX standard only requires
=
to be a comparison operator, not==
; see pubs.opengroup.org/onlinepubs/9699919799/utilities/test.html Mar 30, 2016 at 15:44 -
6Also,
grep 'string' &>/dev/null
is both non-POSIX compliant and much slower to execute (ifstring
appears early in a long output stream) thangrep -q string
. [The caveat there is if you want to be suresomecommand
keeps running even after emittingstring
, in which case usinggrep -q
-- by closing its stdin and exiting after the first instance ofstring
is seen -- can be counterproductive]. (Re: "non-POSIX-compliant",&>
is an extension -- see pubs.opengroup.org/onlinepubs/009695399/utilities/… describing the POSIX-mandated redirection support). Mar 30, 2016 at 15:46 -
1Would help if it was explained why that works/what each parameter does, to encourage full understanding of the syntax– redfox05Oct 17, 2017 at 19:52
-
See also Why is testing “$?” to see if a command succeeded or not, an anti-pattern?– tripleeeDec 15, 2020 at 12:05
-
Another option is to check for regular expression match on the command output.
For example:
[[ "$(./somecommand)" =~ "sub string" ]] && echo "Output includes 'sub string'"
A clean if
/else
conditional shell script:
if ./somecommand | grep -q 'some string'
then
echo "exists"
else
echo "doesn't exist"
fi
-
This seems to be a restatement of earlier answers, except the trivially obvious
else
addition.– tripleeeMay 4 at 17:08
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.
-
3That was the missing piece I needed. Commands like resize2fs need it (2>&1). Thanks for clarifying! 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-– F1LinuxDec 28, 2021 at 20:06