what wrong with this script?

[faust2004]

Hi, friends
I exec follow script
control=$(cat /tmp/scripts/numero_lineas.log |wc -l)
actual=$(cat /tmp/scripts/numero_lineas.txt)
if [ $control -gt $actual ];then
echo "New lines in logfile"
echo $control > /tmp/scripts/numero_lineas.txt
fi
but error appear:: [3]: test: The specified number is not valid for this command.
how to do with it?

Thanks
Sunny

[Stefan Farrelly · Accepted solution]

You forgot the |wc -l on the line starting;
actual=
It should look like;
actual=$(cat /tmp/scripts/numero_lineas.txt|wc -l)

Then the test (if ..-gt) will work as its comparing 2 numbers.

[Jonathan Baker]

It depends whats in /tmp/scripts/numero_lineas.txt. My guess is thatyou're trying to compare a number $control with a string $actual which will be the contents of /tmp/scripts/numero_lineas.txt. It may be that you just need to add | wc -l to the command for $actual e.g. actual=$(cat /tmp/scripts/numero_lineas.txt | wc -l)

[Stefan Farrelly]

I can see what youre doing now, the actual line is supposed to equal what is in the file numero_lineas.txt (which should contain 1 line which has a number). However, when you do your first test does this file have any content ? It should contain only 1 line with a number in it, eg. echo 1 > numero_lineas.txt

[Yogeeraj_1]

hello sunny,

i have one script that does something similar.
(see attachment)

Hope it helps

Cheers
Yogeeraj