Karl Vogel <vogelke+deb...@pobox.com> 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 debian-user-requ...@lists.debian.org 
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/tlnai7x1ch....@news.roaima.co.uk

Reply via email to