From 7c8e096a621d7f569a37903f06b3be99f2ddd376 Mon Sep 17 00:00:00 2001
From: "Alexandre \"kAworu\" Perrin" <kaworu@kaworu.ch>
Date: Mon, 11 Oct 2010 17:00:11 +0200
Subject: [PATCH] avoid using seq in awsetbg, and test for columns count
 argument of -m option.

Signed-off-by: Uli Schlachter <psychon@znc.in>
---
 utils/awsetbg | 23 +++++++++++++++--------
 1 file changed, 15 insertions(+), 8 deletions(-)

diff --git a/utils/awsetbg b/utils/awsetbg
index 0f6386360..bc833c37d 100755
--- a/utils/awsetbg
+++ b/utils/awsetbg
@@ -218,14 +218,21 @@ while [ $# -gt 0 ]; do
             montage=`which montage`
             if test ! -z "$montage"; then
                 shift
-                cols="$1"; shift
-                fns=""
-                for i in `seq $cols`; do
-                    fns="$fns $1";
-                done;
-                shift
-                $montage $fns -geometry +0+0 -tile ${cols}x $CONFIG_DIR/montage.png
-                wallpaper=$CONFIG_DIR/montage.png
+                if [ $# -eq 0 ]; then
+                    echo "-m option need a columns argument"
+                    exit # no more args
+                else
+                    cols="$1"; shift
+                    fns=""
+                    i=$cols
+                    while [ $i -gt 0 ]; do
+                        fns="$fns $1"
+                        i=$(($i - 1))
+                    done
+                    shift
+                    $montage $fns -geometry +0+0 -tile ${cols}x $CONFIG_DIR/montage.png
+                    wallpaper=$CONFIG_DIR/montage.png
+                fi
             else
                 shift
                 echo "montage not found. Falling back to single image"