[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