If doesn't work at bash script

Question

This is not the complete code, 2 first lines are only for "debug" purpose, I'm new on bash scripting and I don't know why if ls $ACTUAL/${BACKUP_DIR}/${NOMBRE}_*.tgz | wc -l returns 2 ,the script execute the then clause...

COPIAS=$(ls $ACTUAL/${BACKUP_DIR}/${NOMBRE}_*.tgz | wc -l)
echo ${COPIAS}
if [ $(ls $ACTUAL/${BACKUP_DIR}/${NOMBRE}_*.tgz | wc -l)>5 ]
    then
        echo "more than 5"
    else
        echo "5 or less"
fi

Dont parse the output from ls.

SMA
– SMA

2015-03-07 15:52:56 +00:00
Commented Mar 7, 2015 at 15:52 — SMA
– SMA, Commented Mar 7, 2015 at 15:52
Take a look at shellcheck.net

Cyrus
– Cyrus

2015-03-07 15:53:16 +00:00
Commented Mar 7, 2015 at 15:53 — Cyrus
– Cyrus, Commented Mar 7, 2015 at 15:53
related serverfault.com/questions/477503/…

Nikos M.
– Nikos M.

2015-03-07 15:55:18 +00:00
Commented Mar 7, 2015 at 15:55 — Nikos M.
– Nikos M., Commented Mar 7, 2015 at 15:55

user4098326 · Accepted Answer · 2015-03-07 16:02:01Z

2

[ is a normal command, so > is treated as a redirection. You can use [[ (which is a keyword) instead. Alternatively, you can escape > to \> so it isn't treated as a redirection.

Beware that > is for string comparison. To compare two numbers you would use -gt.

edited Mar 7, 2015 at 16:02

answered Mar 7, 2015 at 15:55

user4098326

1,7524 gold badges16 silver badges20 bronze badges

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

Comments

Cyrus · Accepted Answer · 2015-03-07 16:57:05Z

1

 array=("$ACTUAL/${BACKUP_DIR}/${NOMBRE}_"*.tgz)

 if [[ ${#array[@]} -gt 5 ]]  # -gt: arithmetic operator greater-than
 then
   echo "more than 5"
 else
   echo "5 or less"
 fi

edited Mar 7, 2015 at 16:57

answered Mar 7, 2015 at 16:30

Cyrus

90.2k15 gold badges112 silver badges173 bronze badges

Collectives™ on Stack Overflow

If doesn't work at bash script

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related