execute mailx using shell script

Question

I am trying to run the below script.

val = `wc -l /home/validate.bad | awk '{print $1}' | tail -n1`
valCount = `wc -l /home/validation.txt | awk '{print $1}'`
if [ "$val" -gt 1 ] && ["$valCount" -gt 1]
then
    mailx -s "Validation failed" -r [email protected] [email protected]<<-EOF
Hi ,
Validation has failed. Please check.
EOF
elif [ "$valCount" -gt 1 ]
then
    mailx -s "Validation pass" -r [email protected] [email protected]<<-EOF
Hi Team,
Validation success.
EOF
fi

But I am getting this error.

Error:
val: comand not found
valCount: command not found
line 3[: : integer expression expected

Assuming this is bash: You have spaces around the assignment operator. Use val= and valCount= respectively. — Robert
– Robert, Commented Jul 28, 2016 at 3:20

sjsam · Accepted Answer · 2016-07-28 04:59:25Z

1

You can't have spaces around = :

val = `wc -l /home/validate.bad | awk '{print $1}' | t` # wrong

and should have been

val=`wc -l /home/validate.bad | awk '{print $1}' | t`

or preferrably

val=$(wc -l </home/validate.bad) 
#`..` is legacy , $() supports nesting, one good reason to go for it
# You use awk and tail uselessly

Also

["$valCount" -gt 1]

should have been

[ "$valCount" -gt 1 ] # mind the spaces for the test constructie
# [spaceSTUFFspace] is the correct form

Sidenote

You may use [ shellcheck ] to check your scripts.

edited Jul 28, 2016 at 4:59

answered Jul 28, 2016 at 4:49

sjsam

22k6 gold badges63 silver badges114 bronze badges

Sign up to request clarification or add additional context in comments.

Collectives™ on Stack Overflow

execute mailx using shell script

1 Answer 1

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Related