hello.

can someone please help me find out what is wrong with this if-then statement? this is a bourne shell script.

if [ w -h | grep $uname | wc -l = 0 ]; then
exit 0
fi

thank you for your help

I assume you have defined $uname before this line else grep has no idea what it is.

That is not a good way to do what you need.

how about this? You may want add the -w grep option depending on your requirements. This can prevent false positives. See the man page for details.

if w -h | grep -q $uname 
then
   exit 0
fi

w -h |grep -q $uname || exit 0

if [ `w -h | grep root | wc -l` -eq 0 ];then
 echo "do something"
fi

thank you frank_rizzo & ghostdog74 syntax-wise the code works. but...the wc is counting 1 when its supposed to count 0. this code is a part of a script that is supposed to weed out invalid users. if the $uname is invalid then there will be no processes running under that user. i thought of adding grep -v grep and the count is now 0. but that doesn't make sense.

thanks again.

there is no reason to count the lines. you really only care if there is a match right?

try this

if w -h | grep -q -w  ^$uname 
then
   exit 0
fi