Applied.

Dan,

Please send me a patch to CREDITS with your info.

Thanks, -Brian


Daniel Widyono wrote:

>Handle --server and --image options.  If --server provided, forward to
>updateclient.
>
>If --clients-file not provided, don't die.
>
>Correct erroneous array variable names (hostnames -> hostnames_and_images).
>
>Regards,
>Dan W.
>  
>
>------------------------------------------------------------------------
>
>--- /usr/sbin/si_pushupdate.orig       2005-07-01 00:33:56.000000000 -0400
>+++ /usr/sbin/si_pushupdate    2005-07-12 13:33:48.735346350 -0400
>@@ -67,6 +67,8 @@
>     "help"                      => \my $help,
>     "version"                   => \my $version,
>     "client=s"                  => \my $base_host_name,
>+    "server=s"                        => \my $imageserver,
>+    "image=s"                 => \my $image,
>     "range=s"                   => \my $range,
>     "domain=s"                  => \my $domain_name,
>     "clients-file=s"            => \my $clients_file,
>@@ -94,6 +96,7 @@
> if($imageserver) {
>     $_ = $imageserver;
>     if(/^-/) { die "\n$program_name: Server name can\'t start with a 
> hyphen.\n$get_help\n\n"; }
>+    $updateclient_options .= " --server $imageserver";
> }
> 
> # must have some specifier for processing files
>@@ -129,15 +132,15 @@
> }
> 
> # setup @hostname array for processing
>-if (-e $clients_file) {
>+if (defined $clients_file && -e $clients_file) {
>     open (FH, "< $clients_file") or die "\n$program_name: Unable to open 
> $clients_file: $!\n";
>     @hostnames_and_images = <FH>;
>     close (FH) or die "\n$program_name: Unable to close $hostfile: $!\n";
>     my (@fields, $testline, $i);
>-    for ($i = 0; $i <= $#hostnames; $i++) {
>+    for ($i = 0; $i <= $#hostnames_and_images; $i++) {
>         # attempt to parse each line, just to make sure there is an image or 
> script specified
>-        chomp($hostnames[$i]);
>-        @fields = split(" ", $hostnames[$i]);
>+        chomp($hostnames_and_images[$i]);
>+        @fields = split(" ", $hostnames_and_images[$i]);
>         if(!$fields[1]) {
>             die "No image/script defined for $fields[0] on line $i of file 
> $clients_file\n";
>         }
>@@ -179,9 +182,9 @@
>         my $node_number;
>         my $count = 0;
>         foreach $node_number ($starting_number .. $ending_number) {
>-            $hostnames[$count] = $base_host_name . $node_number;
>+            $hostnames_and_images[$count] = $base_host_name . $node_number;
>             if ($domain_name) {
>-                $hostnames[$count] = "$hostnames[$count]." .  $domain_name .  
>" $image";
>+                $hostnames_and_images[$count] = 
>"$hostnames_and_images[$count]." .  $domain_name .  " $image";
>             }
>             $count++;
>         }
>  
>

-- 
------------------------------------------------------
 Brian Elliott Finley           Mobile:  630.631.6621
 gpg --keyserver wwwkeys.pgp.net --recv-keys 10F8EE52
------------------------------------------------------



-------------------------------------------------------
This SF.Net email is sponsored by the 'Do More With Dual!' webinar happening
July 14 at 8am PDT/11am EDT. We invite you to explore the latest in dual
core and dual graphics technology at this free one hour event hosted by HP,
AMD, and NVIDIA.  To register visit http://www.hp.com/go/dualwebinar
_______________________________________________
Sisuite-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/sisuite-devel

Reply via email to