Changeset 17253

Timestamp:

2008-09-11T15:30:18+12:00 (16 years ago)

Author:

kjdon

Message:

added use strict; check whether site is non-empty not just defined to test for gs3

File:

• gsdl/trunk/bin/script/build (modified) (4 diffs)

gsdl/trunk/bin/script/build

@@ -35 +35 @@
 package build;
 
+use strict;
+no strict 'refs';
+
 use FileHandle;
 use File::Copy;
@@ -65 +68 @@
     $path_separator . $ENV{'PATH'};
 
+# all the input option variables
+my ($optionfile, $indextype, $append, $manifest, $remove_archives, $remove_import, $buildtype, $maxdocs, @download, $collectdir, $site, $dontinstall, $save_archives, $out, $make_writable, $log_events, $event_log_file, $email_events, $mail_server, $statsfile, $event_header);
+
 &parse_args (\@ARGV);
 
@@ -93 +99 @@
 
 my $cdir = $collectdir;
-if (defined $site)
+if (defined $site && $site =~ /\w/)
 {
     die "GSDL3HOME not set." unless $ENV{'GSDL3HOME'};
@@ -235 +241 @@
     }
     
-    foreach $download_dir (@download) {
+    foreach my $download_dir (@download) {
 
         # remove any leading or trailing whitespace from filenames (just in case)