[LNT] r195506 - By popular demand, when LNT nt fails while running a subprocess command, make sure the error message which is printed has the name of the log file which contains the output of the failing command.

[Chris Matthews]

Author: cmatthews
Date: Fri Nov 22 16:17:56 2013
New Revision: 195506

URL: http://llvm.org/viewvc/llvm-project?rev=195506&view=rev
Log:
By popular demand, when LNT nt fails while running a subprocess command, make sure the error message which is printed has the name of the log file which contains the output of the failing command.

Modified:
    lnt/trunk/lnt/tests/nt.py

Modified: lnt/trunk/lnt/tests/nt.py
URL: http://llvm.org/viewvc/llvm-project/lnt/trunk/lnt/tests/nt.py?rev=195506&r1=195505&r2=195506&view=diff
==============================================================================
--- lnt/trunk/lnt/tests/nt.py (original)
+++ lnt/trunk/lnt/tests/nt.py Fri Nov 22 16:17:56 2013
@@ -600,6 +600,8 @@ def execute_nt_tests(test_log, make_vari
       print >>sys.stderr, '%s: building "nightly tests" with -j%u...' % (
           timestamp(), config.build_threads)
       res = execute_command(test_log, basedir, args, report_dir)
+      if res != 0:
+          print >> sys.std, "Failure while running make build!  See log: %s"%(test_log.name)
 
     # Then 'make report'.
     args = common_args + ['-j', str(config.threads),
@@ -617,6 +619,9 @@ def execute_nt_tests(test_log, make_vari
 
     res = execute_command(test_log, basedir, args, report_dir)
 
+    if res != 0:
+        print >> sys.std, "Failure while running nightly tests!  See log: %s"%(test_log.name)
+
 def load_nt_report_file(report_path, config):
     # Compute the test samples to report.
     sample_keys = []
@@ -781,7 +786,7 @@ def update_tools(make_variables, config,
                           args, config.report_dir)
     build_tools_log.close()
     if res != 0:
-        fatal('unable to build tools, aborting!')
+        fatal('Unable to build tools, aborting! See log: %s'%(build_tools_log_path))
 
 def configure_test_suite(config, iteration):
     """Run configure on the test suite."""
@@ -811,7 +816,7 @@ def configure_test_suite(config, iterati
     res = execute_command(configure_log, basedir, args, config.report_dir)
     configure_log.close()
     if res != 0:
-        fatal('configure failed, log is here: %r' % configure_log_path)
+        fatal('Configure failed, log is here: %r' % configure_log_path)
 
 def copy_missing_makefiles(config, basedir):
     """When running with only_test something, makefiles will be missing,