current community

your communities

more stack exchange communities

Stack Exchange
tour help
careers 2.0
Take the 2-minute tour ×
Stack Overflow is a question and answer site for professional and enthusiast programmers. It's 100% free, no registration required.
up vote 0 down vote favorite

I am trying to check a string that is output from a program, if the string matches a certain content, the while-loop will stop the program. At the same time, I need to count how many times the program has run:

x = "Lookup success"  # this is supposed to be the output from the program
INTERVAL=0  # count the number of runs

while ["$x" != "Lookup failed"]   # only break out the while loop when "Lookup failed" happened in the program
do
   echo "not failed"     # do something
   $x = "Lookup failed"     # just for testing the break-out
   INTERVAL=(( $INTERVAL + 10 )); # the interval increments by 10  
done

echo $x
echo $INTERVAL

But this shell script is not working, with this error:

./test.sh: line 9: x: command not found 
./test.sh: line 12: [[: command not found

Could someone help me please? I appreciate your help.

share|improve this question
    
Kindly accept or upvote the answers if it helped you. –  Anubhab Dec 9 '13 at 6:37

3 Answers 3

up vote 1 down vote

Add a space after [ and before ].

Also, as Jonathan said, you cannot have space in assignments as well.

share|improve this answer
up vote 2 down vote

You need spaces around the [ command name. You also need a space before the ] argument at the end of the command.

You also cannot have spaces around assignments in shell. And your assignment in the loop does not need a $ at the start.

x="Lookup success"
INTERVAL=0  # count the number of runs

while [ "$x" != "Lookup failed" ]
do
    echo "not failed"
    x="Lookup failed"
    INTERVAL=(( $INTERVAL + 10 ))  
done

echo $x
echo $INTERVAL
share|improve this answer
up vote 0 down vote

Not sure if there's a shell that would accept INTERVAL=((...)); my version of ksh and bash on two platforms does not. INTERVAL=$((...)) does work:

#!/bin/bash

x="Lookup success"
INTERVAL=0  # count the number of runs

while [ "$x" != "Lookup failed" ]
do
    echo "not failed"
    x="Lookup failed"
    INTERVAL=$(( $INTERVAL + 10 ))  
done

echo $x
echo $INTERVAL

Credits go to @JonathanLeffler. I'll appreciate up-votes so that next time I don't have to copy-paste others' solution for pointing out a simple typo (comment rights start with rep>=50).

share|improve this answer

Your Answer

 
discard

By posting your answer, you agree to the privacy policy and terms of service.

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