Karl Vogel <[email protected]> wrote:
>      for file in $(ls $MAGDIR/*.[Zz][Ii][Pp] 2> /dev/null); do  ...

mkdir $MAGDIR/silly.zip
touch $MAGDIR/silly.zip/not-a-zip-file

If you're going to insist on using "ls" you should consider "ls -d".

Personally, I'd still go for this construct:

    for FILE in "$MAGDIR"/*.[Zz][Ii][Pp]
    do
        test -f "$FILE" || continue
        ...

But each to our own.
Chris


-- 
To UNSUBSCRIBE, email to [email protected] 
with a subject of "unsubscribe". Trouble? Contact [email protected]
Archive: http://lists.debian.org/[email protected]

Reply via email to