changed the upgrade system to die after fatal errors

docs/changelog/6.x.x.txt

@@ -96,4 +96,6 @@ webgui.
  - Added a view privileges setting to each forum.
  - Bugfix [ 822805 ] Emptying trash can cause fatal error
    (Thanks to Steve Simms.)
+ - Changed the upgrade system to stop after an error so the sys admin can fix
+   the problem before continuing.
 

docs/upgrades/upgrade_5.9.9-6.0.0.pl

@@ -1,5 +1,4 @@
 #!/usr/bin/perl
-
 use lib "../../lib";
 use File::Path;
 use Getopt::Long;

sbin/upgrade.pl

@@ -221,6 +221,7 @@ foreach my $config (keys %config) {
 				print "OK\n" unless ($quiet);
 			} else {
 				print "Failed!\n" unless ($quiet);
+				fatalError();
 			}
 		}
 		print "\tUpgrading to ".$upgrade{$upgrade}{to}."..." unless ($quiet);
@@ -232,12 +233,14 @@ foreach my $config (keys %config) {
 			print "OK\n" unless ($quiet);
 		} else {
                 	print "Failed!\n" unless ($quiet);
+			fatalError();
                 }
 		if ($upgrade{$upgrade}{pl} ne "") {
 			my $cmd = $perl." ".$upgrade{$upgrade}{pl}." --configFile=".$config;
 			$cmd .= " --quiet" if ($quiet);
 			if (system($cmd)) {
 				print "\tProcessing upgrade executable failed!\n";
+				fatalError();
 			}
 		}
 		$config{$config}{version} = $upgrade{$upgrade}{to};
@@ -291,6 +294,18 @@ sub checkVersion {
 	}
 }
 
+#-----------------------------------------
+sub fatalError {
+	print <<STOP;
+
+The upgrade process failed and has stopped so you can either restore
+from backup, or attempt to fix the problem and continue.
+
+STOP
+	exit;
+}
+
+
 #-----------------------------------------
 sub _parseDSN {
     my($dsn, $args) = @_;