#At file:///home/msvensson/mysql/5.1-rpl-latest/
2663 Magnus Svensson 2008-08-29
Print error message in diff_files if any of the two files to diff does not exist
modified:
client/mysqltest.cc
mysql-test/mysql-test-run.pl
=== modified file 'client/mysqltest.cc'
--- a/client/mysqltest.cc 2008-08-09 12:59:57 +0000
+++ b/client/mysqltest.cc 2008-08-29 10:25:19 +0000
@@ -3335,6 +3335,14 @@ void do_diff_files(struct st_command *co
sizeof(diff_file_args)/sizeof(struct command_arg),
' ');
+ if (access(ds_filename.str, F_OK) != 0)
+ die("command \"diff_files\" failed, file '%s' does not exist",
+ ds_filename.str);
+
+ if (access(ds_filename2.str, F_OK) != 0)
+ die("command \"diff_files\" failed, file '%s' does not exist",
+ ds_filename2.str);
+
if ((error= compare_files(ds_filename.str, ds_filename2.str)))
{
/* Compare of the two files failed, append them to output
=== modified file 'mysql-test/mysql-test-run.pl'
--- a/mysql-test/mysql-test-run.pl 2008-08-11 08:41:12 +0000
+++ b/mysql-test/mysql-test-run.pl 2008-08-29 10:25:19 +0000
@@ -3486,12 +3486,21 @@ sub report_failure_and_restart ($) {
if ( !defined $tinfo->{logfile} )
{
my $logfile= $path_current_testlog;
- if ( defined $logfile and -f $logfile )
+ if ( defined $logfile )
{
- # Test failure was detected by test tool and its report
- # about what failed has been saved to file. Save the report
- # in tinfo
- $tinfo->{logfile}= mtr_fromfile($logfile);
+ if ( -f $logfile )
+ {
+ # Test failure was detected by test tool and its report
+ # about what failed has been saved to file. Save the report
+ # in tinfo
+ $tinfo->{logfile}= mtr_fromfile($logfile);
+ }
+ else
+ {
+ # The test tool report didn't exist, display an
+ # error message
+ $tinfo->{logfile}= "Could not open test tool report '$logfile'";
+ }
}
}
| Thread |
|---|
| • bzr commit into mysql-5.1 branch (msvensson:2663) | Magnus Svensson | 29 Aug 2008 |