debian-manage-mirror script - todo

"Raman Pandarinathan" <[email protected]>
Newsgroups gmane.linux.terminal-server.devel
Message-ID <[email protected]>
Dear developers

 While looking through the plugin source codes, I saw a TODO marked in
 manage-mirror script for debian.

 I have modified the code for arbitary number of mirrors, which seems
to work for me.

I am submitting the patch here.

 Raman.P

-------------------patch to manage mirror script--------------------
 --- manage-mirror.orig	
+++ /usr/share/ltsp/plugins/ltsp-build-client/Debian/010-manage-mirror	
@@ -49,13 +49,15 @@
                 # only one mirror specified
                 add_mirror "$mirror_list"
             else
-                # TODO: support an arbitrary number of mirrors
-                for number in 1 2 3 4 5 6 7 8 9 ; do
-                    mirror="$(echo "$mirror_list" | cut -d , -f $number)"
-                    if [ -n "$mirror" ]; then
-                        add_mirror "$mirror"
-                    fi
-                done
+		OIFS=$IFS
+		IFS=,
+		for mirror in $mirror_list; do
+			if [ -n "$mirror" ]; then
+				add_mirror "$mirror"
+			fi
+		done
+		IFS=$OIFS
+			
             fi
         }

-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/
_____________________________________________________________________
Ltsp-developer mailing list.   To un-subscribe, or change prefs, goto:
      https://lists.sourceforge.net/lists/listinfo/ltsp-developer
For additional LTSP help,   try #ltsp channel on irc.freenode.net
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.