[LNT] r198455 - Handle the case of a missing configure file in the	test suite better.
    Chris Matthews 
    cmatthews5 at apple.com
       
    Fri Jan  3 14:50:37 PST 2014
    
    
  
Author: cmatthews
Date: Fri Jan  3 16:50:37 2014
New Revision: 198455
URL: http://llvm.org/viewvc/llvm-project?rev=198455&view=rev
Log:
Handle the case of a missing configure file in the test suite better.
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=198455&r1=198454&r2=198455&view=diff
==============================================================================
--- lnt/trunk/lnt/tests/nt.py (original)
+++ lnt/trunk/lnt/tests/nt.py Fri Jan  3 16:50:37 2014
@@ -815,8 +815,14 @@ def configure_test_suite(config, iterati
     configure_log.flush()
 
     print >>sys.stderr, '%s: configuring...' % timestamp()
-    res = execute_command(configure_log, basedir, args, config.report_dir, config.verbose)
-    configure_log.close()
+    try:
+        res = execute_command(configure_log, basedir, args,
+                              config.report_dir, config.verbose)
+    except OSError as e:
+        fatal('Configure failed to execute, '
+              'log is here: %r\nError: %r' % configure_log_path, e)
+    finally:
+        configure_log.close()
     if res != 0:
         fatal('Configure failed, log is here: %r' % configure_log_path)
 
    
    
More information about the llvm-commits
mailing list